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

Fix Actions not running on master branch #5011

Merged

Conversation

TPGamesNL
Copy link
Member

Description

Fixes the tests not running on the master branch, when a commit is made. This is caused by #4946, which mistakenly used the branch name main instead of master, but we don't have such a branch.


Target Minecraft Versions: any
Requirements: none
Related Issues: #4946 (non-fixing)

@TPGamesNL TPGamesNL added bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. unit testing For issues/PRs related to the Skript unit testing system. labels Aug 5, 2022
@TheLimeGlass
Copy link
Contributor

We are a pull request only push system right now, so there isn't really a need to waste our 2000 monthly minutes on running the same test twice.

I think it's better to have them disabled on the master branch. The pull request check is enough.

@TPGamesNL
Copy link
Member Author

TPGamesNL commented Aug 5, 2022

The 2000 minutes a month is only for private repos, it doesn't count for this repo
image

Besides, the master branch adds that all users can download a nightly build from the latest commit instead of having to build themselves

@TheLimeGlass TheLimeGlass merged commit c0b8e95 into SkriptLang:master Aug 6, 2022
@TPGamesNL TPGamesNL deleted the fix/actions-not-on-master branch August 13, 2022 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. unit testing For issues/PRs related to the Skript unit testing system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants