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.

Sign up for GitHub

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

Jump to bottom

Update to .NET 8 #40

Merged
merged 4 commits into from
Aug 21, 2024
Merged

Update to .NET 8 #40

merged 4 commits into from
Aug 21, 2024

Conversation

oskogstad
Copy link
Contributor

Description

Updates project to .NET 8

Verification

  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)
  • All tests run green

Documentation

  • User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

@SandGrainOne
Copy link
Member

Would it be an option to target both .NET 6 and .NET 8?

@oskogstad
Copy link
Contributor Author

oskogstad commented Jun 12, 2024 •
edited
Loading

Would it be an option to target both .NET 6 and .NET 8?

Hm, yeah!
Does that mean we don't have to make a new major version for this NuGet?
This also updates all the dependencies that has security vulnerabilities, #39 will be redundant
all other PRs except #38 and #36 will be redundant

@oskogstad oskogstad marked this pull request as ready for review June 12, 2024 13:27
@elsand elsand merged commit 8e7a5e7 into Altinn:main Aug 21, 2024
knuhau added a commit to knuhau/altinn-apiclient-maskinporten that referenced this pull request Oct 22, 2024
@knuhau
Revert "Update to .NET 8 (Altinn#40)"
1d697e0 
This reverts commit 8e7a5e7.
elsand pushed a commit that referenced this pull request Oct 22, 2024
@knuhau
fix: Add yt01 token exchange, revert .NET 8 upgrade (#46)
7319e73 
* Revert "Update to .NET 8 (#40)"

This reverts commit 8e7a5e7.

* add yt01 token exchange
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@elsand elsand elsand approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@oskogstad @SandGrainOne @elsand