Skip to content

Refactor Github Actions#264

Merged
deathaxe merged 8 commits intomasterfrom
feat/refactor-actions
Apr 6, 2024
Merged

Refactor Github Actions#264
deathaxe merged 8 commits intomasterfrom
feat/refactor-actions

Conversation

@deathaxe
Copy link
Copy Markdown
Member

@deathaxe deathaxe commented Apr 6, 2024

No description provided.

@deathaxe deathaxe force-pushed the feat/refactor-actions branch 13 times, most recently from 0da402f to 464db7f Compare April 6, 2024 15:47
deathaxe added 2 commits April 6, 2024 18:03
They are no longer used.

1. To generate code coverage reports latest available coverage package
   is installed as library by ST (requires Package Control!). 
   Coverage reports are created from within ST's plugin environment.
2. Coverage reports are to be uploaded via via codecov action, which has to
   deal with installing required coverage python packages itself.
Package Control is configured not to auto-update or touch any package.
Restoring something after installing Package Control is obsolete.
@deathaxe deathaxe force-pushed the feat/refactor-actions branch 9 times, most recently from 1ea6e64 to 2b7b46a Compare April 6, 2024 18:25
deathaxe added 5 commits April 6, 2024 20:37
Starting and stopping ST is just a waste of time.
It will be started to install Package Control.
1. Allowing to not install UnitTesting is odd as tests won't run without it.
2. Installing a version which is unrelated with current action revision is
   also odd and confusing. If a certain version is desired for CI, it can be
   selected via `SublimeText/UnitTesting/actions/setup@<revision>`.

The `setup` action always installs the UnitTesting package, which is bundled
with the action's revision, if UnitTesting is not the checked out package or
hasn't been installed as extra-package. That's possible as the action is part
of UnitTesting package. The package doesn't need to be re-downloaded.
@deathaxe deathaxe force-pushed the feat/refactor-actions branch from 2b7b46a to 99f2489 Compare April 6, 2024 18:41
This commit prevents Package Control.sublime-package from being downloaded
and installed, if Package Control folder already exists as extracted package.

This may be the case if it is the package to test or was installed
via `extra-packages` input.
@deathaxe deathaxe merged commit ccda2b4 into master Apr 6, 2024
@deathaxe deathaxe deleted the feat/refactor-actions branch April 6, 2024 19:06
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.

1 participant