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.

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

Migration Guide for NUnit 4.0 #827

Merged
merged 19 commits into from
Nov 7, 2023
Merged

Migration Guide for NUnit 4.0 #827

merged 19 commits into from
Nov 7, 2023

Conversation

manfred-brands
Copy link
Member

Fixes #825

@manfred-brands
Copy link
Member Author

@SeanKilleen I used ~/images for references to .png files as I saw this being done in other place, but these don't work in github mark-down. There is one existing image reference in assertions.md that works in github.
It would be nice if it works there for all, but I don't know how that affects your deployment.

@Arhell Arhell requested a review from SeanKilleen November 5, 2023 00:17
@SeanKilleen
Copy link
Member

SeanKilleen commented Nov 6, 2023

Beginning to review this now; will need to continue it tonight. @manfred-brands we can't guarantee that image references will work in GitHub markdown. The true way to know if something will work is to serve the contents via docfx (for convenience, you could open the PR in a GitHub codespace and then type serve and that should fire it up. No worries though, I'll check it and change as needed.

(Update: pics look fine to me)

@SeanKilleen
Copy link
Member

@manfred-brands @OsirisTerje how do we feel about possibly creating a "Migration Guide" in general and counting 3.x -> 4.x as a topic within it? I'm thinking this would then be appended or preprended to for future migration guidance that will inevitably occur. If you agree I'll make the changes.

Copy link
Member

@SeanKilleen SeanKilleen left a comment

Choose a reason for hiding this comment

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

(Edit: Moved comment to review pane for better discussion)

@manfred-brands
Copy link
Member Author

manfred-brands commented Nov 7, 2023

we can't guarantee that image references will work in GitHub markdown

My question was about the relative path used. The images that use ../../images would work, but the ~/images not as in the repository the images directory is not in the root, but under docs. If the relative links work in docfx, I would prefer to use those so users can look in github.

@manfred-brands
Copy link
Member Author

how do we feel about possibly creating a "Migration Guide" in general

We should only need one for major (and incompatible releases).
I don't mind either way, as long as the most resent one is on top so user don't have to scroll over no longer relevant items.

@SeanKilleen
Copy link
Member

@manfred-brands

If the relative links work in docfx, I would prefer to use those so users can look in github.

Ah, I see! Makes sense. I'll fiddle with that a bit.

Maybe a lint setting for a maximum line length would be good.

In the docs, I've taken to keeping things on one line until they are separate thoughts/components, since they render the same. I could be a little more consistent, but that's the idea. Does that cause any difficulty for you that I might be missing?

@SeanKilleen
Copy link
Member

@manfred-brands image paths updated and appear to be shown correctly in both my docfx preview and GitHub markdown. Thanks for that; I'll apply it elsewhere.

@manfred-brands
Copy link
Member Author

In the docs, I've taken to keeping things on one line until they are separate thoughts/components, since they render the same. I could be a little more consistent, but that's the idea. Does that cause any difficulty for you that I might be missing?

Not all software wraps long lines:

Github Extensions:
image

Visual Studio:
image

Even though github (online) wraps lines, diffs are on a per line basis. If such a line is extremely long it look like a big change.

I prefer a reasonable maximum line length somehow.

@SeanKilleen
Copy link
Member

@manfred-brands

I prefer a reasonable maximum line length somehow.

Created #830 to discuss. Input welcome there! It never bothered me; I just scrolled to the right as needed. But I can definitely see the value of doing it.

Copy link
Member

@SeanKilleen SeanKilleen 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 getting this, @manfred-brands! Really nice work. Merging now.

@SeanKilleen SeanKilleen merged commit d68b9b4 into master Nov 7, 2023
7 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 7, 2023
* Migration Guide for NUnit 4.0

* Linting fixes

* Remove wrong example of a 3rd party library depending on nunit

* Add TOC entry for article

* Small phrasing update

* Change to note formatting

* minor word change

* Fix typo

* minor formatting

* NUnit4 -> NUnit 4.x

* Update link name.

* Minor word choice

* Add space

* Update other image paths

* Turn into non-version-specific migration guide

* Add link in NUnit 4 article

* formatting updates

* Fix note formatting

---------

Co-authored-by: Sean Killeen <[email protected]> d68b9b4
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.

Create migration guide for upgrading to v4
4 participants