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

#311 prerelease deployment #312

Closed
wants to merge 20 commits into from
Closed

Conversation

michaelwittwer
Copy link
Member

@michaelwittwer michaelwittwer commented Jun 10, 2020

this pull request contains two main features:

  • Replace travis build with github-action build

  • To be able to release preview functionality via npm we can update the .releaserc.json file to allow a branch to be published, this process is for now not tied to the creation of a PR for a certain branch.
    This would be the desired behavior, but for now we just allow a more manual approach.

    The channel defines the npm dist-tag and the prerelease defines the v5.8.0-pr311.1 (bold part), those two values should always be equal.

    With this setup consumers are able to install prerelease version using the command npm i @shiftcode/dynamo-easy@pr311

@coveralls
Copy link

coveralls commented Jun 10, 2020

Coverage Status

Coverage remained the same at 95.61% when pulling 72058fa on #311-prerelease-deployment into 3ffded4 on master.

configure #311-prerelease-deployment branch to be published on channel pr311
@michaelwittwer michaelwittwer self-assigned this Jun 10, 2020
@michaelwittwer michaelwittwer linked an issue Jun 10, 2020 that may be closed by this pull request
@michaelwittwer michaelwittwer requested review from simonmumenthaler and removed request for simonmumenthaler June 10, 2020 14:27
@michaelwittwer michaelwittwer marked this pull request as draft June 10, 2020 14:53
@michaelwittwer michaelwittwer marked this pull request as ready for review June 10, 2020 14:54
Copy link
Contributor

@simonmumenthaler simonmumenthaler left a comment

Choose a reason for hiding this comment

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

since it is not a feature - and no new version should be published: move changes to another branch without 'feat' commits...

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.

publish features from PR branches to npm
3 participants