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

Automate Browserlist DB Update #783

Merged

Conversation

abdullahwaheed
Copy link
Contributor

@abdullahwaheed abdullahwaheed commented Oct 5, 2022

Ticket

Automate browserslist DB update

What has changed

Added a Github workflow that would run periodically(weekly), update browserslist DB, create a PR and merge it automatically.
Used a shared workflow for this

Dependent on #599

@abdullahwaheed abdullahwaheed marked this pull request as ready for review December 20, 2022 14:26
uses: openedx/.github/.github/workflows/update-browserslist-db.yml@master

secrets:
requirements_bot_github_token: ${{ secrets.requirements_bot_github_token }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are we using requirement bot here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

its an edX bot account that Arch-BOM and Arbi-BOM are using to create automated PRs.

Copy link
Contributor

@arbrandes arbrandes left a comment

Choose a reason for hiding this comment

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

I'm approving this as core contributor. It's been open long enough.

@codecov
Copy link

codecov bot commented May 23, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (9e59252) 66.43% compared to head (448f677) 66.43%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #783   +/-   ##
=======================================
  Coverage   66.43%   66.43%           
=======================================
  Files         127      127           
  Lines        3158     3158           
  Branches      911      911           
=======================================
  Hits         2098     2098           
  Misses       1012     1012           
  Partials       48       48           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@arbrandes arbrandes merged commit 1e634e7 into openedx:master May 23, 2023
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