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(ci): pre-release dependabot #401

Merged
merged 13 commits into from
Jan 28, 2021
Merged

chore(ci): pre-release dependabot #401

merged 13 commits into from
Jan 28, 2021

Conversation

TarikGul
Copy link
Member

@TarikGul TarikGul commented Jan 25, 2021

This PR builds on issue #380 .

This PR has changed from creating a automated publishing pipeline to a sole Dependabot script to create PR's every Monday with updated packaging from @polkadot/...

After some discussion and thought, it would make sense to do each section of automated publishing in seperate PR's and to build out the process piece by piece.


 | Every Monday at XX:XX(am/pm) check for updates in the following 3 packages |
 |        1                   2                       3                       |
 | @polkadot/api => @polkadot/apps-config => @polkadot/util-crypto.           |
 | When the updates exists, create the PR.                                    |
  

@TarikGul TarikGul changed the title Automated publishing CI/CD: automated publishing Jan 25, 2021
@TarikGul TarikGul changed the title CI/CD: automated publishing chore(ci): automated publishing Jan 26, 2021
@TarikGul TarikGul changed the title chore(ci): automated publishing chore(ci): pre-release dependabot Jan 26, 2021
TarikGul and others added 4 commits January 26, 2021 17:14
feat: automate npm publishing

fix: filename

fix: cleanup, add ignores

fix: revert back a single dependabot script

feat: add reviewers, change prefix to fit convetional commiting

fix: remove ignores
Copy link
Contributor

@dvdplm dvdplm left a comment

Choose a reason for hiding this comment

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

Nice, it's more succinct than I thought it'd be.

cc @niklasad1 Maybe we can apply the same technique to other places? Nothing specific comes to mind, but good to know it's there. :)

# PST timezone
timezone: "America/Los_Angeles"
rebase-strategy: "auto"
commit-message:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
commit-message:
open-pull-requests-limit: 20
commit-message:

Copy link
Contributor

@TriplEight TriplEight Jan 28, 2021

Choose a reason for hiding this comment

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

In case you don't want to be spammed with these PRs, feel free to change the number or not use it.

Copy link
Member Author

Choose a reason for hiding this comment

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

Good call, I'll definitely add it in, for now I think 10 will do. I saw that 5 is the default so I think having a little more room for open pull requests would definitely be beneficial. Thanks Denis.

@TarikGul TarikGul merged commit 5390aa7 into master Jan 28, 2021
@TarikGul TarikGul deleted the automated-publishing branch January 28, 2021 17:09
TarikGul added a commit that referenced this pull request Jan 28, 2021
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.

4 participants