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

IOS-10444 Mistica importer from GH Actions #390

Merged
merged 4 commits into from
Jul 31, 2024

Conversation

amegias
Copy link
Contributor

@amegias amegias commented Jul 23, 2024

🎟️ Jira ticket

https://jira.tid.es/browse/IOS-10444

πŸ₯… What's the goal?

Having the generation fixed here #387, we need to integrate it in GitHub Actions.

🚧 How do we do it?

Now it's simpler since it's just a node execution, see the code.
I've added the tests execution too.

πŸ§ͺ How can I verify this?

Tests

I've forced it manually:

Generation

I've forced it manually:

@amegias amegias changed the title Mistica importer ghaction Mistica importer from GH Actions Jul 23, 2024
@amegias amegias changed the base branch from main to mistica-importer July 23, 2024 10:25
@amegias amegias force-pushed the mistica-importer-ghaction branch from b7f666f to 2c1e100 Compare July 23, 2024 10:25
@amegias amegias force-pushed the mistica-importer-ghaction branch from b3dc354 to a545cf2 Compare July 23, 2024 11:11
@amegias amegias changed the title Mistica importer from GH Actions IOS-10444 Mistica importer from GH Actions Jul 23, 2024
@amegias amegias marked this pull request as ready for review July 23, 2024 11:15
runs-on: self-hosted-novum-mac
build:
name: Build & generate tokens
runs-on: ubuntu-latest
Copy link
Contributor Author

Choose a reason for hiding this comment

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

mac is not needed!

Comment on lines +5 to +7
push:
paths:
- 'scripts/mistica-skin-generator/**'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

only when the generator changes

@amegias amegias merged commit c092de5 into mistica-importer Jul 31, 2024
1 check passed
@amegias amegias deleted the mistica-importer-ghaction branch July 31, 2024 09:47
amegias added a commit that referenced this pull request Jul 31, 2024
…387)

* Tool

* Test tokens generation

* Revert "Test tokens generation"

This reverts commit 5d236ea.

* CR improvements

* Use snapshots

* Add JSDoc

* setup eslint

* configure prettier

* Run Prettier

* Fix lint

* IOS-10444 Mistica importer from GH Actions (#390)

* Tests action for the tool

* Create a new workflow for mistica-skin-generator-tests and modify the generation flow to generate tokens

* Test show URL

* Clean up

---------

Co-authored-by: Abel Toledano <[email protected]>
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.

3 participants