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

Browserslist DB bump script using wrong branch #212

Open
doamatto opened this issue Jan 10, 2025 · 0 comments
Open

Browserslist DB bump script using wrong branch #212

doamatto opened this issue Jan 10, 2025 · 0 comments
Labels
code Related to PrivacySpy's source code dependencies Pull requests that update a dependency file good first issue Good for newcomers problem Something isn't right

Comments

@doamatto
Copy link
Collaborator

What's the issue? Describe the problem below.
#211 shows commits from #171 being proposed for merge.

What did you expect to be the case instead?
The action (.github/workflows/browserslist.yml) should checkout master and bump the database there.

When it's applied to a MR, it should checkout that MR and merge into that MR. A good solution could be simply specifying that only master, like so :

on:
  push:
    branches:
      - main

Screenshots?
N/A

Additional context?
N/A

@doamatto doamatto added problem Something isn't right good first issue Good for newcomers code Related to PrivacySpy's source code dependencies Pull requests that update a dependency file labels Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code Related to PrivacySpy's source code dependencies Pull requests that update a dependency file good first issue Good for newcomers problem Something isn't right
Projects
None yet
Development

No branches or pull requests

1 participant