Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

.jshintignore, .jshintrc #22

Open
bchurchill opened this issue Mar 6, 2017 · 8 comments
Open

.jshintignore, .jshintrc #22

bchurchill opened this issue Mar 6, 2017 · 8 comments

Comments

@bchurchill
Copy link
Owner

After running npm run clean-build and npm run create-build there were two new files created, .jshintignore and .jshintrc. I don't know the best practice for handling these files in version control (or what they do, frankly), so perhaps we can decide if they should be added to the .gitignore or committed into the repository?

@bchurchill
Copy link
Owner Author

@brenmcnamara I intended to assign this to you but it doesn't look like you're a collaborator yet. To make that happen check this link: https://github.com/bchurchill/badi-cal/invitations.

@brettz9
Copy link

brettz9 commented Jun 18, 2017

.jshintrc includes the instructions on how to Run the Commands (RC) of JSHint, a linting project which I think has fallen out of favor a bit with the predominance now of ESLint. .jshintignore should indicate the files or directories that should be exempt from JSHint linting (e.g., node_modules).

Since you're already using ESLint, I don't think you actually want those files created, and from a quick glance at your code, I don't see how they were created as it seems it should be making ESLint files instead.

@brettz9
Copy link

brettz9 commented Jun 18, 2017

Btw, feel free to start a wiki page on @bahaidev listing Baha'i-related repos such as this one (if not the repo itself).

@brettz9
Copy link

brettz9 commented Jun 18, 2017

Btw, +9 for your project's use of ES6/Babel!

@bchurchill
Copy link
Owner Author

You can thank @brenmcnamara for that! I'm a foreigner in these vast and spacious lands of javascript...

@brenmcnamara
Copy link
Collaborator

Sorry for responding so late. I'm not sure how they got created but you don't want those files. We are using ESLint. These files are for configuring rules that check for javascript code quality. I can try to see why these files were created.

This isn't a breaking issue though. Whether you commit those files or not, things should work fine.

@brenmcnamara
Copy link
Collaborator

@brettz9 - It doesn't look like the Baha'i Dev organization has any owners. How can we add or get access to repos? Thanks!

@brettz9
Copy link

brettz9 commented Jun 18, 2017

When logging on recently, Github suggested renaming the "Owners" team since that team is no longer used to determine ownership (that is instead now handled by "roles").

I did just now add a "Collaborators" team as a general umbrella group, however, and added my TextBrowser repo to it (though if our numbers start getting unwieldy (a welcome development!), I might change to have teams for each repository along with any teams that combine repository access for related projects or teams for long-time contributing team members).

For future reference, as long is one is already a member of the organization (in this case, bahaidev), one can ask to join any (non-"secret") team in the organization.

But in this case, I've already added you both as members to that umbrella team (though in your case, @brenmcnamara , you'll first need to agree to join the organization, but after that, you should already have access to the group, and thus to TextBrowser (and any other repos you might wish to add for that team)).

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

No branches or pull requests

3 participants