Skip to content
This repository has been archived by the owner on Mar 23, 2023. It is now read-only.

ci: draft releases when merging to master #1693

Merged
merged 10 commits into from
Feb 21, 2020
Merged

Conversation

faustbrian
Copy link
Contributor

@faustbrian faustbrian commented Feb 21, 2020

Drafts a new release based on the version property in the package.json file. The changelog is currently not automatically created but could be added if needed.

It's important to use the versioning commands of npm or yarn to ensure that the workflows and artifacts are properly versioned. Manually editing files should be avoided as those commands exist for a reason.

This intentionally only runs on a push to master as what is merged should be 100% working based on tests and builds on develop passing.

@ghost ghost added Complexity: Medium Less than 256 lines changed. Status: In Progress The issue or pull request is being worked on. labels Feb 21, 2020
@codecov
Copy link

codecov bot commented Feb 21, 2020

Codecov Report

Merging #1693 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1693   +/-   ##
========================================
  Coverage    63.87%   63.87%           
========================================
  Files          139      139           
  Lines         3815     3815           
  Branches       797      797           
========================================
  Hits          2437     2437           
  Misses        1152     1152           
  Partials       226      226

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 699d99c...d6df4a5. Read the comment docs.

@faustbrian faustbrian force-pushed the workflow-release branch 2 times, most recently from bad59e7 to c028314 Compare February 21, 2020 11:12
@faustbrian faustbrian marked this pull request as ready for review February 21, 2020 11:21
@ghost ghost added Status: Needs Review The issue or pull request needs a review by a developer of the team. and removed Status: In Progress The issue or pull request is being worked on. labels Feb 21, 2020
@alexbarnsley alexbarnsley merged commit db72bb3 into develop Feb 21, 2020
@ghost ghost deleted the workflow-release branch February 21, 2020 17:53
@ghost ghost removed the Status: Needs Review The issue or pull request needs a review by a developer of the team. label Feb 21, 2020
@dated
Copy link
Contributor

dated commented Feb 21, 2020

unfortunately this breaks builds for win and macos on forks since the build steps depend on github secrets which, well... are secret.

@faustbrian
Copy link
Contributor Author

You'll still be busy another 8 weeks with tests so don't act like it's urgent @dated :trollface:

@dated
Copy link
Contributor

dated commented Feb 22, 2020

:cripes:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Complexity: Medium Less than 256 lines changed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants