Skip to content

feat: fix CI by upgrading Node version#220

Merged
civsiv merged 1 commit intomasterfrom
feature/fix-ci
Mar 14, 2024
Merged

feat: fix CI by upgrading Node version#220
civsiv merged 1 commit intomasterfrom
feature/fix-ci

Conversation

@civsiv
Copy link
Copy Markdown
Contributor

@civsiv civsiv commented Mar 14, 2024

Before:
Screenshot 2024-03-14 at 12 42 36

After (Test Suite successfully installs):

Screenshot 2024-03-14 at 12 43 51

with:
dotnet-version: 3.1.419
- name: Setup Node.js 14.x
- name: Setup Node.js 18.17.1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why 18.17.1 rather than 18.19.1? Is it what other OA apps are using?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes exactly

@civsiv civsiv merged commit 163c655 into master Mar 14, 2024
@civsiv civsiv deleted the feature/fix-ci branch March 14, 2024 14:16
@civsiv civsiv linked an issue Mar 14, 2024 that may be closed by this pull request
civsiv added a commit that referenced this pull request Mar 15, 2024
civsiv added a commit that referenced this pull request Mar 19, 2024
* feat: add workflows/README.md to elaborate on GitHub workflows

* feat: fix CI by upgrading Node version (#220)

* move most of the detail into the workflows

* Update .github/workflows/README.md

Co-authored-by: Luke Winship <luke.winship@gmail.com>

* Update .github/workflows/README.md

Co-authored-by: Luke Winship <luke.winship@gmail.com>

* Update .github/workflows/README.md

Co-authored-by: Luke Winship <luke.winship@gmail.com>

* Update .github/workflows/README.md

Co-authored-by: Luke Winship <luke.winship@gmail.com>

* review comments

* fix conflict

---------

Co-authored-by: Luke Winship <luke.winship@gmail.com>
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.

Fix out of date Node version in OpenActive.Server.NET CI workflows

2 participants