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

Set the string flavour tag of the jar in the actions artifacts to nightly #4994

Merged
merged 8 commits into from
Dec 27, 2022

Conversation

TheLimeGlass
Copy link
Collaborator

@TheLimeGlass TheLimeGlass commented Jul 30, 2022

Description

Set the string flavour tag of the jar in the actions artifacts to nightly as per @APickledWalrus

@AyhamAl-Ali AyhamAl-Ali added the enhancement Feature request, an issue about something that could be improved, or a PR improving something. label Jul 30, 2022
Copy link
Member

@APickledWalrus APickledWalrus left a comment

Choose a reason for hiding this comment

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

Maybe we should just have tests generate a jar named Skript-nightly? Would remove the need for most of the changes here 🙂

@TheLimeGlass
Copy link
Collaborator Author

TheLimeGlass commented Aug 15, 2022

Maybe we should just have tests generate a jar named Skript-nightly? Would remove the need for most of the changes here 🙂

Already said in Discord that if the name was Skript-nightly every reference in the Skript code and Skript test runner would need to be changed. I did try that, and it still errored for some reason because the name was Skript-nightly. So for Skript's sake, it only uses Skript.jar, and after the tests are ran, it can be named whatever, and to whatever we decide on later down the road.

@TPGamesNL
Copy link
Member

I don't think the tests should be a nightly tasks by default, those are reserved for GH Actions only (IMO we should actually only have them for the ones on the master branch, and a diff flavour for other branches).

I think it'd be better to first run the skriptTest task, than clean nightlyRelease

@TheLimeGlass
Copy link
Collaborator Author

TheLimeGlass commented Aug 16, 2022

I don't think the tests should be a nightly tasks by default, those are reserved for GH Actions only (IMO we should actually only have them for the ones on the master branch, and a diff flavour for other branches).

I think it'd be better to first run the skriptTest task, than clean nightlyRelease

It was that originally, but Pickle thought otherwise, so it's changed. We need to come up with an agreement.

I think it's perfect at it's current state.

@TheLimeGlass
Copy link
Collaborator Author

TheLimeGlass commented Aug 17, 2022

I made it so both the Skript-test.jar and Skript-nightly.jar are uploaded in the artifact. It's good to include the actual jar that was used in the tests.

It's Skript-test.jar because in this pull request #4979 I added JUnit dependencies inside the jar, so it's got different classes inside. Skript-nightly is a snapshot of the Skript release at that time, aka a nightly build.

build.gradle Outdated Show resolved Hide resolved
.github/workflows/java-8-builds.yml Outdated Show resolved Hide resolved
@TheLimeGlass TheLimeGlass added the unit testing For issues/PRs related to the Skript unit testing system. label Dec 16, 2022
@TheLimeGlass TheLimeGlass removed the request for review from Moderocky December 27, 2022 09:41
@TheLimeGlass TheLimeGlass merged commit b14a3cd into SkriptLang:master Dec 27, 2022
@TheLimeGlass TheLimeGlass deleted the change-nightly branch December 27, 2022 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature request, an issue about something that could be improved, or a PR improving something. 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