Skip to content

feat: update to angular v20#507

Merged
ebrehault merged 1 commit intoguillotinaweb:masterfrom
krishna217:master
Nov 17, 2025
Merged

feat: update to angular v20#507
ebrehault merged 1 commit intoguillotinaweb:masterfrom
krishna217:master

Conversation

@krishna217
Copy link
Copy Markdown
Contributor

No description provided.

@krishna217
Copy link
Copy Markdown
Contributor Author

CI build is failing. @ebrehault could u take a look ? Thanks

@ebrehault
Copy link
Copy Markdown
Collaborator

@krishna217 Hi!

I think that's probably because actions/setup-node@v1 is obsolete.

Go to .github/workflows/ci.yml and .github/workflows/deploy.yml and replace

uses: actions/setup-node@v1

with

uses: actions/setup-node@v3

@krishna217 krishna217 force-pushed the master branch 2 times, most recently from 7fe5ec6 to 48e5217 Compare November 17, 2025 11:17
@krishna217
Copy link
Copy Markdown
Contributor Author

@krishna217 Hi!

I think that's probably because actions/setup-node@v1 is obsolete.

Go to .github/workflows/ci.yml and .github/workflows/deploy.yml and replace

uses: actions/setup-node@v1

with

uses: actions/setup-node@v3

Thanks, fixed it

@ebrehault
Copy link
Copy Markdown
Collaborator

@krishna217 Looks good :)

Could you please update the package version in projects/schema-form/package.json (make it 2.14.0), that way, when merging the PR, the new version will be released automatically on npm (and also add an entry in CHANGELOG.md).
Thanks :)

@krishna217
Copy link
Copy Markdown
Contributor Author

@krishna217 Looks good :)

Could you please update the package version in projects/schema-form/package.json (make it 2.14.0), that way, when merging the PR, the new version will be released automatically on npm (and also add an entry in CHANGELOG.md). Thanks :)

done

@ebrehault ebrehault merged commit 9c4f213 into guillotinaweb:master Nov 17, 2025
1 check passed
@ebrehault
Copy link
Copy Markdown
Collaborator

Thank you so much!!!!

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