Skip to content

Add public Architecture enum value for s390x#52906

Merged
akoeplinger merged 1 commit intodotnet:mainfrom
uweigand:s390x-architecture
May 19, 2021
Merged

Add public Architecture enum value for s390x#52906
akoeplinger merged 1 commit intodotnet:mainfrom
uweigand:s390x-architecture

Conversation

@uweigand
Copy link
Copy Markdown
Contributor

  • Add S390x value to System.Runtime.InteropServices.Architecture enum

  • Add native s390x architecture detection

Note to reviewers: This adds a new value to a publicly visible enum, so in that sense it affects the .NET API. I don't know if there are any special requirements when making such changes.

* Add S390x value to System.Runtime.InteropServices.Architecture enum

* Add native s390x architecture detection
@ghost
Copy link
Copy Markdown

ghost commented May 18, 2021

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

@filipnavara
Copy link
Copy Markdown
Member

filipnavara commented May 18, 2021

I don't know if there are any special requirements when making such changes.

I think you need to file an issue first and it has to go through API review

@uweigand
Copy link
Copy Markdown
Contributor Author

I don't know if there are any special requirements when making such changes.

I think you need to file an issue first and it has to go through API review

Thanks! I've filed an issue now.

@marek-safar marek-safar added the blocked Issue/PR is blocked on something - see comments label May 19, 2021
@akoeplinger
Copy link
Copy Markdown
Member

API was approved in #52909

@akoeplinger akoeplinger merged commit dc088c3 into dotnet:main May 19, 2021
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
@uweigand uweigand deleted the s390x-architecture branch May 27, 2021 11:00
@ghost ghost locked as resolved and limited conversation to collaborators Jun 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants