-
Notifications
You must be signed in to change notification settings - Fork 25.3k
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.
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
Replace M.A.All meta-package with Microsoft.AspNetCore.App - MD files only #5314
Comments
For reference: aspnet/Announcements#287 |
@guardrex client-side\spa-services.md isn't project.json-based. You can remove that one from your list above, if that's what it's tracking. |
@guardrex I looked at the environments and error-handling docs and I don't see any references to "Microsoft.AspNetCore.All" in those. |
[2X EDIT 😄] "Replace ALL meta-package with App Microsoft.AspNetCore.App - MD files only" ... I missed the "MD files only" bit. I'm pulling the data for when we have an issue that pertains to samples. |
Oh geez I forgot to look at the samples and hit just the docs. |
I totally missed what @Rick-Anderson was asking for here. I thought we'd do both ... fix the doc AND fix the sample. My bad. I think the samps will be a different issue. When we set that up tho ... I'll paste that table back in. [EDIT] As this issue stands, I think only one of us really needs to breeze thru and fix the MD files. Really shouldn't take all that long (guessing). |
@guardrex what's the state of this issue? |
I was working the update for topics+samples simultaneously; but as u know, 2.1 sample updates on #5495 have been slow. Would you like me to run thru and get the last few of these out of the topics (and we'll get the samples done later)? |
@guardrex no hurry, just wanted to get the state. It's not a priority. |
ok ... I'll get it. It's going to take a while longer to get samples squared away, so we may as well go ahead. I think (hope) that less than a dozen topics need a touch to drop it out of the text (with versioning, of course). The samples associated with those topics may still use the |
@Rick-Anderson @scottaddie @tdykstra I'm ready to proceed on this issue today, but let me get the word on if there was a design change. The last info that I have is no version on the App package ... |
Ok ... we're good ... no version for the time being ... #9129 (comment) |
currently mentioned in 22 files. This issue is about updating the docs that specifically mention Microsoft.AspNetCore.All
We need to convert a few of these and agree on the language and style of the update.
Updating the samples that use the ALL meta-package is a lower priority. See Update all 2.0 samples to use M.A.App, aka Microsoft.AspNetCore.App #6382
The text was updated successfully, but these errors were encountered: