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

Explains how protected internal works with the virtual and override keywords in different assemblies. #39192

Merged
merged 4 commits into from
Jan 19, 2024
Merged

Explains how protected internal works with the virtual and override keywords in different assemblies. #39192

merged 4 commits into from
Jan 19, 2024

Conversation

PowerfulBacon
Copy link
Contributor

@PowerfulBacon PowerfulBacon commented Jan 18, 2024 •
edited by github-actions bot
Loading

Summary

Adds an explanation of how the protected internal accessibility modifier operates when overriding a member defined in some base class defined in another assembly.

  • If the base class is defined in the same assembly, the accessibility modifiers of an overriden member needs to be protected internal.
  • If the base class is defined in another assembly, the accessibility modifiers of an overriden member needs to be protected only.

Internal previews

📄 File 🔗 Preview link
docs/csharp/language-reference/keywords/protected-internal.md protected internal (C# Reference)

@ghost ghost added the community-contribution Indicates PR is created by someone from the .NET community. label Jan 18, 2024
@PowerfulBacon
Copy link
Contributor Author

@dotnet-policy-service agree

BillWagner
BillWagner requested changes Jan 18, 2024
View reviewed changes
Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

Thanks for making these additions @PowerfulBacon

I have some language suggestions for you to consider. Once we resolve them, this is ready to merge.

docs/csharp/language-reference/keywords/protected-internal.md Outdated Show resolved Hide resolved
docs/csharp/language-reference/keywords/protected-internal.md Outdated Show resolved Hide resolved
@BillWagner BillWagner enabled auto-merge (squash) January 19, 2024 15:41
@BillWagner BillWagner merged commit 91188a1 into dotnet:main Jan 19, 2024
8 checks passed
BillWagner added a commit that referenced this pull request Jan 25, 2024
@BillWagner @dotnet-policy-service
@jeffhandley @gewarren @PowerfulBacon @azure-sdk @mrhaugan @hoyosjs @drognanar @Nikanoras @BartoszKlonowski @ReubenBond @IEvangelist @Maoni0 @mikekistler @monty241 @github-actions @JamesNK @MarcoRossignoli @dependabot @surgicalcoder @adegeo @Evangelink
Publish to live (#39258)
1ffea8a 
* Add prIssueManagement.yml to onboard repo to GitOps.ResourceManagement as FabricBot replacement

 Details on the replacement service and the syntax of the new yaml configuration file is available publicly at: https://microsoft.github.io/GitOps/policies/resource-management.html

Please review and merge this PR to complete the process of onboarding to the new service.

* Deleting fabricbot.json

* Fill in missing scheduled search description. De-dupe Admin/Write permission check.

* .NET 9 Preview 1 breaking changes (#39198)

* Explains how protected internal works with the virtual and override keywords in different assemblies. (#39192)

* Update protected-internal.md

* Updates the sub-header

* Makes the comment line length shorter

* Apply suggestions from code review

---------

Co-authored-by: Bill Wagner <[email protected]>

* Update package index with latest published versions (#39207)

* Add third option for collection expressions analyzers (#39199)

* Update package index with latest published versions (#39209)

* Add assembly attributes using project file (#39208)

* Update types.md (#39214)

There was some duplicate(?) code, demonstrating the creation of two Point instances twice. Surely, no need for this?

* Remove unnecessary section on getting fx symbols for perfcollect (#39200)

* Remove unnecessary section on getting fx symbols for perfcollect

* Remove hanging link

* Explain how to run MSTestRunner tests in VsCode (#39155)

* Explain how to run MSTestRunner tests in VsCode

* Add the run MSTestRunner with VSCode section

* Changes to MSTestRunner document

- Change TIP to a NOTE
- Fix header title

* Fix compilation error for AppLogEvents example (#39206)

* Use 'entry' word instead of 'key' for TLS in network programming (#39139)

* Clarify stateless worker routing rules (#39210)

* Clarify stateless worker routing rules

* Update date

* Update docs/orleans/grains/stateless-worker-grains.md

---------

Co-authored-by: David Pine <[email protected]>

* When I made the fix in this PR: #38713 I didn't notice one additional place that needed to be changed (#38964)

Co-authored-by: Maoni0 <[email protected]>

* Create webapi app rather than webapp. (#39098)

* Explain how to solve "It was not possible to find any compatible fram… (#38937)

* Explain how to solve "It was not possible to find any compatible framework version."

Occurs when DOTNET_ROOT is pointing to another install such as 5.0.23, but the dotnet-dump expects a more recent release such as 6 or 7.

* Update docs/core/diagnostics/faq-dumps.yml

Co-authored-by: Genevieve Warren <[email protected]>

---------

Co-authored-by: David Pine <[email protected]>
Co-authored-by: Genevieve Warren <[email protected]>

* .NET dependabot-bot automated PR. (#39213)

Co-authored-by: BillWagner <[email protected]>

* Update package index with latest published versions (#39217)

* Update package index with latest published versions (#39218)

* Reduce metric name text size (#39211)

* Update package index with latest published versions (#39220)

* Update diagnostic default verbosity (#39215)

Co-authored-by: Marco Rossignoli <[email protected]>

* Correct the typo: TreatWarningsAsErrors -> WarningsAsError (#39219)

* Update package index with latest published versions (#39225)

* Update package index with latest published versions (#39229)

* Bump Microsoft.ML.TensorFlow and Microsoft.ML

Bumps [Microsoft.ML.TensorFlow](https://github.com/dotnet/machinelearning) and [Microsoft.ML](https://github.com/dotnet/machinelearning). These dependencies needed to be updated together.

Updates `Microsoft.ML.TensorFlow` from 3.0.0 to 3.0.1
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

Updates `Microsoft.ML` from 3.0.0 to 3.0.1
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

---
updated-dependencies:
- dependency-name: Microsoft.ML.TensorFlow
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: Microsoft.ML
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump System.Linq.Dynamic.Core

Bumps [System.Linq.Dynamic.Core](https://github.com/zzzprojects/System.Linq.Dynamic.Core) from 1.3.7 to 1.3.8.
- [Release notes](https://github.com/zzzprojects/System.Linq.Dynamic.Core/releases)
- [Changelog](https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/master/CHANGELOG.md)
- [Commits](zzzprojects/System.Linq.Dynamic.Core@v1.3.7...v1.3.8)

---
updated-dependencies:
- dependency-name: System.Linq.Dynamic.Core
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump Microsoft.ML.ImageAnalytics

Bumps [Microsoft.ML.ImageAnalytics](https://github.com/dotnet/machinelearning) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

---
updated-dependencies:
- dependency-name: Microsoft.ML.ImageAnalytics
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump actions/upload-artifact from 4.1.0 to 4.3.0

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.1.0 to 4.3.0.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@1eb3cb2...26f96df)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump Microsoft.ML.FastTree and Microsoft.ML

Bumps [Microsoft.ML.FastTree](https://github.com/dotnet/machinelearning) and [Microsoft.ML](https://github.com/dotnet/machinelearning). These dependencies needed to be updated together.

Updates `Microsoft.ML.FastTree` from 3.0.0 to 3.0.1
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

Updates `Microsoft.ML` from 3.0.0 to 3.0.1
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

---
updated-dependencies:
- dependency-name: Microsoft.ML.FastTree
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: Microsoft.ML
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump System.Linq.Dynamic.Core

Bumps [System.Linq.Dynamic.Core](https://github.com/zzzprojects/System.Linq.Dynamic.Core) from 1.3.7 to 1.3.8.
- [Release notes](https://github.com/zzzprojects/System.Linq.Dynamic.Core/releases)
- [Changelog](https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/master/CHANGELOG.md)
- [Commits](zzzprojects/System.Linq.Dynamic.Core@v1.3.7...v1.3.8)

---
updated-dependencies:
- dependency-name: System.Linq.Dynamic.Core
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump Microsoft.ML.Recommender and Microsoft.ML

Bumps [Microsoft.ML.Recommender](https://github.com/dotnet/machinelearning) and [Microsoft.ML](https://github.com/dotnet/machinelearning). These dependencies needed to be updated together.

Updates `Microsoft.ML.Recommender` from 0.21.0 to 0.21.1
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

Updates `Microsoft.ML` from 3.0.0 to 3.0.1
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

---
updated-dependencies:
- dependency-name: Microsoft.ML.Recommender
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: Microsoft.ML
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump Microsoft.ML

Bumps [Microsoft.ML](https://github.com/dotnet/machinelearning) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

---
updated-dependencies:
- dependency-name: Microsoft.ML
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump Microsoft.ML.TimeSeries and Microsoft.ML

Bumps [Microsoft.ML.TimeSeries](https://github.com/dotnet/machinelearning) and [Microsoft.ML](https://github.com/dotnet/machinelearning). These dependencies needed to be updated together.

Updates `Microsoft.ML.TimeSeries` from 3.0.0 to 3.0.1
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

Updates `Microsoft.ML` from 3.0.0 to 3.0.1
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

---
updated-dependencies:
- dependency-name: Microsoft.ML.TimeSeries
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: Microsoft.ML
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump Microsoft.ML

Bumps [Microsoft.ML](https://github.com/dotnet/machinelearning) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

---
updated-dependencies:
- dependency-name: Microsoft.ML
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump Microsoft.ML

Bumps [Microsoft.ML](https://github.com/dotnet/machinelearning) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

---
updated-dependencies:
- dependency-name: Microsoft.ML
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump Microsoft.ML

Bumps [Microsoft.ML](https://github.com/dotnet/machinelearning) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

---
updated-dependencies:
- dependency-name: Microsoft.ML
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump Microsoft.ML

Bumps [Microsoft.ML](https://github.com/dotnet/machinelearning) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/dotnet/machinelearning/releases)
- [Commits](https://github.com/dotnet/machinelearning/commits)

---
updated-dependencies:
- dependency-name: Microsoft.ML
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* fix template name (#39252)

Fixes #39237

When I renamed the template to put it last in the list, I didn't update the config option. This fixes that.

* Erroneous semicolon causes CS1513 compile error (#39251)

* Add some clarifications around C++/CLI projects (#39228)

* Update article

* Apply suggestions from code review

Co-authored-by: Genevieve Warren <[email protected]>

* minor

---------

Co-authored-by: Genevieve Warren <[email protected]>

* Correct ubuntu and debian dependencies (#39253)

* Update linux-ubuntu.md

* Update linux-debian.md

* Update linux-ubuntu.md

* Update linux-ubuntu-2204.md

* Update linux-ubuntu-2004.md

* Update linux-ubuntu-2210.md

* Update linux-ubuntu-2304.md

* Update linux-ubuntu-2310.md

* Update package index with latest published versions (#39255)

* Update package index with latest published versions (#39256)

* Options validate onstart (#39254)

* Touch ms.date

* Add validate on start details

* Update MSTest runner extensions info (#39152)

* Update MSTest runner extensions info

- List licensing model for each extension
- Fix Code Coverage command line options
- Use released versions

* Apply suggestions from code review

* More coverage documentation

* Update docs/core/testing/unit-testing-mstest-runner-extensions.md

* Update docs/core/testing/unit-testing-mstest-runner-extensions.md

* Apply suggestions from code review

* Update extensions

* Fix broken links

---------

Co-authored-by: Jakub Chocholowicz <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dotnet-policy-service[bot] <123482357+dotnet-policy-service[bot]@users.noreply.github.com>
Co-authored-by: Jeff Handley <[email protected]>
Co-authored-by: Genevieve Warren <[email protected]>
Co-authored-by: PowerfulBacon <[email protected]>
Co-authored-by: Azure SDK Bot <[email protected]>
Co-authored-by: Marius Riis Haugan <[email protected]>
Co-authored-by: Juan Hoyos <[email protected]>
Co-authored-by: Artur Spychaj <[email protected]>
Co-authored-by: Nikanoras Zavadskij <[email protected]>
Co-authored-by: Bartosz Klonowski <[email protected]>
Co-authored-by: Reuben Bond <[email protected]>
Co-authored-by: David Pine <[email protected]>
Co-authored-by: Maoni Stephens <[email protected]>
Co-authored-by: Maoni0 <[email protected]>
Co-authored-by: Mike Kistler <[email protected]>
Co-authored-by: Guido Leenders <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: BillWagner <[email protected]>
Co-authored-by: James Newton-King <[email protected]>
Co-authored-by: Marco Rossignoli <[email protected]>
Co-authored-by: Marco Rossignoli <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: surgicalcoder <[email protected]>
Co-authored-by: Andy (Steve) De George <[email protected]>
Co-authored-by: Amaury Levé <[email protected]>
Co-authored-by: Jakub Chocholowicz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BillWagner BillWagner BillWagner approved these changes

Assignees
No one assigned
Labels
community-contribution Indicates PR is created by someone from the .NET community. dotnet-csharp/svc lang-reference/subsvc
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@PowerfulBacon @BillWagner @dotnet-bot