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

Fixes for PR3638 #3680

Merged
merged 1 commit into from
Jun 6, 2022
Merged

Fixes for PR3638 #3680

merged 1 commit into from
Jun 6, 2022

Conversation

flodolo
Copy link
Collaborator

@flodolo flodolo commented Jun 6, 2022

@bakulf

This is built/open against your current PR. Feel free to close if it's not useful.

You can see the result of the action here
https://github.com/flodolo/mozilla-vpn-client/actions/runs/2447966583

@flodolo flodolo requested a review from bakulf June 6, 2022 13:21
@bakulf bakulf merged commit 8944fa6 into addonTutorial Jun 6, 2022
@bakulf bakulf deleted the addon_fixes branch June 6, 2022 14:36
bakulf added a commit that referenced this pull request Jun 10, 2022
* Generate tutorial addons

* Comment applied

* Removed -g and -t params from generate_ts.sh

* No 'name' var assigned

* Load addons from manifest files

* Split addons by types

* Load and unload guides and tutorials

* Update scripts/utils/generate_strings.py

Co-authored-by: Francesco Lodolo <[email protected]>

* Fixes for PR3638 (#3680)

* Fix the unit-tests

* Add python3-lxml as dep for linux packages

* Fix a missing path

* Fix the path for macos addons

* Fix the strin generation script

* Remove the addon folder

* Addon index

* Github action for addon generation

* Review comments applied

* metavar uppercase

* Removed node for the wasm/addon github action

Co-authored-by: Francesco Lodolo <[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.

2 participants