pester / Pester

Pester is the ubiquitous test and mock framework for PowerShell.
https://pester.dev/
Other
3.08k stars 470 forks source link

Proposed Release Notes for first 4.x RTM Version #853

Closed alx9r closed 7 years ago

alx9r commented 7 years ago

Pursuant to discussion in #811 here is a draft of release notes for whatever the first 4.x RTM version will be ($first4xRTM). These would be the first dedicated release notes for Pester. This will also be the first RTM version that includes the major changes that occurred between versions 3 and 4. Accordingly, the release notes I am proposing should include all the salient user-observable changes between 3.4.6 and $first4xRTM. If I missed anything, please tell me.

I propose that release notes for each version be kept up-to-date on the github releases page.

I also propose that all other mentions of release notes provide only a link to the github releases page. The other places that might mention such release notes include the following:

I think this is better for the following reasons:

  1. Including content in the release notes fields through channels other than the github releases page seems to mean (a possibly unavoidable) risk of those release notes being stale. I don't think that's a good setup considering that important known issues for a release may be published via the release notes well after the release date.
  2. We only have to publish the release notes content in one format. Github uses markdown. Pester module manifests are (probably) just plaintext. Who knows what formatting PowerShell gallery supports. Each format has its own set of quirks and the potential to sap precious developer time.
  3. The release notes content for all other channels is static. This means that the content for those fields can simply be hard-coded and forgotten about.

Proposed Github Releases Page Content

<content>

There is a [guide for migrating from Pester 3 to Pester 4]() to help you understand how the changes from Pester 3 to Pester 4 might affect your tests.

What's New since 3.4.6:

Deprecated starting with 4.0.0:

Removed starting with 4.0.0:

</content>

Proposed Release Notes Content for all Other Places

<content>

Please find the latest release notes for all versions of Pester at https://github.com/pester/Pester/releases

</content>

nohwnd commented 7 years ago

Imho the code coverage output to jacoco is worth mentioning. The packages Pester ships in should also be smaller because all unnecessary files are deleted from them, how much smaller I did not check :) There will very likely be two more breaking changes that are not in the codebase yet: Contains to FileContentMatch, and the Progress reporting will get disabled by default as it has big performance overhead. What we will use to to enable it again I am not sure yet. You can find those issues in milestone 4, if you want to discuss those changes. (Sorry for the lack of formatting, I am writing this from a phone :) )

alx9r commented 7 years ago

Imho the code coverage output to jacoco is worth mentioning.

I omitted this by mistake. I have added it.

The packages Pester ships in should also be smaller because all unnecessary files are deleted from them...

Cool. I did not know about this. I have added this.

There will very likely be two more breaking changes that are not in the codebase yet...

I will add those to the proposed release notes once they are completed.

nohwnd commented 7 years ago

I am drafting the release notes in our releases.

alx9r commented 7 years ago

@nohwnd I don't see 4.0.7 at that URL.

it-praktyk commented 7 years ago

I can see v. 4.0.7

I don't see - it should probably be in part about a nested Describe{} - information that in that case Tag can be used only once. I remember discussions with @dlwyatt that only "more external tag usage is handled".

alx9r commented 7 years ago

@nohwnd Per github documentation:

...Only users with write access to the repository can view drafts of releases...

I'm pretty sure I don't have write access. I assume @it-praktyk does.

nohwnd commented 7 years ago

@alx9r you are now part of our team :) well deserved. Please still use PR to sending changes, instead of committing directly.

nohwnd commented 7 years ago

@it-praktyk added that to the release notes.

alx9r commented 7 years ago

@nohwnd The release notes look good to me. 👍

it-praktyk commented 7 years ago

Added the new Should syntax

We need review about_Should and probably the wiki too.

nohwnd commented 7 years ago

@it-praktyk please review in separate issue.

it-praktyk commented 7 years ago

The related issue #876