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

chore(release): releasing 2.0.0 manually #274

Merged
merged 4 commits into from
Jun 10, 2017
Merged

chore(release): releasing 2.0.0 manually #274

merged 4 commits into from
Jun 10, 2017

Conversation

ta2edchimp
Copy link
Collaborator

Due to a hickup in the automatic release setup, we have to go back to manually releasing (at least version 2.0.0).

@codecov-io
Copy link

codecov-io commented Jun 9, 2017

Codecov Report

Merging #274 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #274   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           9      9           
  Lines         209    209           
=====================================
  Hits          209    209

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a4de6c...52f1b68. Read the comment docs.

@ta2edchimp
Copy link
Collaborator Author

As already mentioned elsewhere, the automatic sematic-release process is broken. To get #237 released (and to eventually get #270 merged and released) I had to do a manual release. Pushing the changes directly is blocked, therefore this PR, which requires a review ( @sarbbottam @ljharb I summon you ... please help — and finally approve 😜 ).

I'd also suggest to make a clean up after bringing #270 out the door, and to stay with manually releasing (as iirc we already concluded) ...

Since we are transpiling into `dist`, that directory s required to be included within the package, but `src` and `test` are not.
Copy link
Collaborator

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

Don't want to block the release, but big -1 on npmignoring src/tests.

node_modules/
.nyc_output/
coverage/
src/
Copy link
Collaborator

Choose a reason for hiding this comment

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

i'm really not a fan of npmignoring src or tests - imo both things should always be published so that npm explore foo && npm install && npm test always works.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That is an absolutely valid point, thank's for bringing that on the table. I'll change that back soon.

@ta2edchimp ta2edchimp merged commit 9c6d090 into sarbbottam:master Jun 10, 2017
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.

None yet

3 participants