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

Ignore certain files when deciding what needs an update. #93

Merged
merged 2 commits into from Apr 20, 2018

Conversation

aomarks
Copy link
Member

@aomarks aomarks commented Apr 10, 2018

Also only allow publishing from master.

Reference: https://github.com/lerna/lerna#lernajson and https://github.com/lerna/lerna#--allow-branch-glob

(not convinced this is working yet)

Also only allow publishing from master.
lerna.json Outdated
"CHANGELOG.md",
".travis.yml",
"appveyor.yml",
".gitignore",
Copy link
Contributor

Choose a reason for hiding this comment

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

we really should just remove all the .gitignores

@aomarks
Copy link
Member Author

aomarks commented Apr 12, 2018

I removed the changelog and test file glob from this list. The idea is that if a dependent has picked up an important bug fix, that should be reflected in the CHANGELOG, and maybe also in their tests, and we would want to trigger a release.

I should also point out that this lerna feature doesn't actually appear to work in my experience so far, and other users seem to be reporting the same thing: lerna/lerna#1357

@aomarks aomarks merged commit d6cd0c3 into master Apr 20, 2018
@aomarks aomarks deleted the lerna-ignore branch April 20, 2018 21:08
aomarks added a commit that referenced this pull request Jul 9, 2018
Bump analyzer; bump test fixture; prepare 1.2.0 release.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants