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

update templates of rebar3 grisp configure for grisp.io #90

Merged
merged 4 commits into from
Nov 13, 2024

Conversation

GwendalLaurent
Copy link
Member

The current version of the templates used by rebar3 grisp configure were not up to date with the latest changes of grisp.io.

  • With the first release of grisp_connect the template can now tag v1.0.0 instead of the main branch
  • grisp_updater_grisp2 wasn't included in the deps if the user decided to create a grisp.io app. It it now the case

How to test ?

  1. Create a dummy rebar3 project
  2. Include rebar3_grisp in the plugin section of the rebar.config file of that project
  3. Create the _checkouts directory in that dummy project
  4. Either clone or create a simlink to this repository
    4.1. Checkout this branch
  5. You can now run rebar3 grisp configure inside your dummy project and generate GRiSP apps

{{/grisp_io_linking}} {{{{name}}}, [
{{/grisp_io_linking}}
{{#grisp_io}} {grisp_updater, [
{system, {grisp_updater_grisp2, #{}}},
Copy link
Contributor

Choose a reason for hiding this comment

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

{signature_check, false},
%{signature_certificates, {priv, grisp_demo, "certificates/updates"}},

@GwendalLaurent GwendalLaurent marked this pull request as ready for review October 11, 2024 17:48
@GwendalLaurent GwendalLaurent merged commit 66c2192 into master Nov 13, 2024
10 checks passed
@GwendalLaurent GwendalLaurent deleted the feature/SEA-308 branch November 13, 2024 08:43
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