Skip to content

adding .github folder and templates - #442

Merged
ekryski merged 2 commits into
masterfrom
github-templates
Oct 22, 2016
Merged

ekryski merged 2 commits into
masterfrom
github-templates

Conversation

@ekryski

@ekryski ekryski commented Oct 22, 2016

Copy link
Copy Markdown
Contributor

@feathersjs/core-team please review and add feedback. We'll iterate in this PR and then can add a script to add this folder to all the other repos as outlined in #435.

Comment thread .github/issue_template.md Outdated
(First please check that this issue is not already solved as [described
here](https://github.com/feathersjs/feathers/blob/master/.github/contributing.md#report-a-bug))

- [ ] Tell us how you broke things. The more detailed the better.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe less accusing? Something like "Tell us what broke."

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually good point

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol true dat

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you break things?!!!

Comment thread .github/issue_template.md Outdated

**NodeJS version**:

**Operating System**: No newline at end of file

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we ask for the client configuration here, too? Browser version and module loader (if applicable)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ya probably. At least browser version and module loader

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very good call, the more detail the better.

@corymsmith

Copy link
Copy Markdown
Contributor

Once we agree on this I can sort the script to add it to all repos, have a good idea on that, should be pretty simple using something like https://github.com/github/hub since I already have all the repos in a folder.

@ekryski

ekryski commented Oct 22, 2016

Copy link
Copy Markdown
Contributor Author

@corymsmith That was my thinking as well. The quick and dirty would be if you already have the folders and write access to simply make a script that rips through all of them and copies the folder to each one, adds it to the end of .npmignore, commits and pushes straight to master.

@corymsmith

Copy link
Copy Markdown
Contributor

Exactly what I was thinking, though I was thinking of PR but thats probably overkill

@ekryski

ekryski commented Oct 22, 2016

Copy link
Copy Markdown
Contributor Author

For this stuff I think so. It doesn't have any impact on code and it's going to be the same in every repo.

@corymsmith

Copy link
Copy Markdown
Contributor

I've got a script sorted for this so let me know when we're good to go

@ekryski
ekryski merged commit b4e224a into master Oct 22, 2016
@ekryski
ekryski deleted the github-templates branch October 22, 2016 21:52
daffl pushed a commit that referenced this pull request Aug 29, 2018
… folder support. (#442)

* Windows compatible version of the original compile comand with public folder support.

- Was `"rm -rf lib/ && babel -d lib/ src/ && mkdir lib/public/ && cp
src/public/* lib/public/"` in legacy but that wasn't windows compatible.
Although `rimraf` takes care of the first half, I suspect the public
copy was lost because a recursive mkdir+cp is hard to find. But `babel
-D` handles that.
- Also fixed the extra `-rf` that should not be used with `rimraf`.

* .gitignore adds for .idea (e.g. WebStorm) and .vs (VS2017) folders

* Oops, src/public only exists in the legacy branch
daffl pushed a commit that referenced this pull request Aug 29, 2018
… folder support. (#442)

* Windows compatible version of the original compile comand with public folder support.

- Was `"rm -rf lib/ && babel -d lib/ src/ && mkdir lib/public/ && cp
src/public/* lib/public/"` in legacy but that wasn't windows compatible.
Although `rimraf` takes care of the first half, I suspect the public
copy was lost because a recursive mkdir+cp is hard to find. But `babel
-D` handles that.
- Also fixed the extra `-rf` that should not be used with `rimraf`.

* .gitignore adds for .idea (e.g. WebStorm) and .vs (VS2017) folders

* Oops, src/public only exists in the legacy branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants