Skip to content

Conversation

nkakouros
Copy link
Contributor

@nkakouros nkakouros commented Apr 1, 2018

Two things here:

  • judging by the "Added ..." lines of the changelog, this justifies pumping to 0.5.0 instead of a patch release, ie 0.4.1

  • I changed the order of mentioning changes, I mention first the new stuff ("Added..."), then the improved stuff and then the bug fixes.

  • I have reviewed the Contributor Guidelines.

  • I have reviewed the Code of Conduct and agree to abide by it

Two things here:

- judging by the "Added ..." lines of the changelog, this justifies pumping to 0.5.0 instead of a patch release, ie 0.4.1
- I changed the order of mentioning changes, I mention first the new stuff ("Added..."), then the improved stuff and then the bug fixes.
@nkakouros nkakouros requested a review from a team as a code owner April 1, 2018 12:55
@sublimino sublimino added this to the v0.5.0 milestone Apr 2, 2018
@sublimino
Copy link
Member

sublimino commented Apr 2, 2018

Awesome @tterranigma, thanks!

Added this to the 0.5.0 milestone as the final step before release.

@btamayo btamayo mentioned this pull request Apr 2, 2018
5 tasks
README.md Outdated
## Version history
## Version

*0.5.0* (April xx, 2018)
Copy link
Contributor

Choose a reason for hiding this comment

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

Looking at June now.

README.md Outdated
* Added support for test scripts that have the [unofficial strict mode](redsymbol.net/articles/unofficial-bash-strict-mode/) enabled.
* Added support for tabs in test names.
* Improved stability on Windows and macOS platforms.
* Fixed an installation bug where if Bats was uploaded from Windows to a server installation would fail ([commit](7afdf57bd3cbd216f6dd98b2112f2cf4fa521870))
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you use the bare commit number here? The link appears broken, and GitHub should auto-link it correctly.

README.md Outdated
* Added support for tabs in test names.
* Improved stability on Windows and macOS platforms.
* Fixed an installation bug where if Bats was uploaded from Windows to a server installation would fail ([commit](7afdf57bd3cbd216f6dd98b2112f2cf4fa521870))
* Fixed TAP compliance for skipped tests
Copy link
Contributor

Choose a reason for hiding this comment

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

Some more items to add:

It might also be worth noting at the top that 0.5.0 generally preserves compatibility with 0.4.0, but with some compatibility improvements and a massive performance boost. In other words:

  • all existing tests should remain compatible
  • tests that might've failed or exhibited unexpected behavior on earlier versions of Bash should now also pass or behave as expected

@ghost ghost assigned sublimino Jun 3, 2018
@ghost ghost added the review label Jun 3, 2018
@sublimino
Copy link
Member

@mbland you've been powering through the backlog, awesome work :D I've updated this PR with your comments.

README.md Outdated
* Workaround for preserving stack info after calling an exported function under Bash < 4.4 (#87)
* Fixed TAP compliance for skipped tests
* Added support for tabs in test names.
* Fixed an installation bug where if Bats was uploaded from Windows to a server installation would fail ([commit](https://github.com/bats-core/bats-core/commit/7afdf57bd3cbd216f6dd98b2112f2cf4fa521870))
Copy link
Contributor

Choose a reason for hiding this comment

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

Ah...actually, I think that bug's finally fixed by #88, wherein I changed bin/bats from a symlink to a wrapper script, given the continued problems from #32.

@nkakouros
Copy link
Contributor Author

Last changes are in place. This looks like a solid changelog. How about tagging 0.5.0? That would be really awesome.. Moving almost!

q.bats Outdated
@@ -0,0 +1,9 @@
#!/usr/bin/env bats
Copy link
Contributor

Choose a reason for hiding this comment

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

This file looks extraneous. Remove it?

README.md Outdated
@@ -384,6 +384,24 @@ on the wiki.

Bats is [SemVer compliant](https://semver.org/).

*0.5.0* (June xx, 2018)
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be 1.0.0 now. Let's go ahead and put June 8, 2018 in there, too. (If we slip, I can tweak it when it's time to tag and release per #16.)

README.md Outdated
@@ -384,6 +384,24 @@ on the wiki.

Bats is [SemVer compliant](https://semver.org/).

*0.5.0* (June xx, 2018)

`0.5.0` generally preserves compatibility with `0.4.0`, but with some Bash compatibility improvements and a massive performance boost. In other words:
Copy link
Contributor

Choose a reason for hiding this comment

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

1.0.0

* Workaround for preserving stack info after calling an exported function under Bash < 4.4 (#87)
* Fixed TAP compliance for skipped tests
* Added support for tabs in test names.
* Fixed an installation bug where if Bats was uploaded from Windows to a server installation would fail (#88)
Copy link
Contributor

Choose a reason for hiding this comment

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

Actually, that would be #91 now. Also it seems it could be worded better, something like: bin/bats and install.sh now work reliably on Windows (#91).

Copy link
Contributor

Choose a reason for hiding this comment

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

I meant that the new line should replace this one, but no matter; I'll delete the old line during the cleanup pass.

* Workaround for preserving stack info after calling an exported function under Bash < 4.4 (#87)
* Fixed TAP compliance for skipped tests
* Added support for tabs in test names.
* Fixed an installation bug where if Bats was uploaded from Windows to a server installation would fail (#88)
Copy link
Contributor

Choose a reason for hiding this comment

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

I meant that the new line should replace this one, but no matter; I'll delete the old line during the cleanup pass.

@mbland mbland merged commit a3057cb into bats-core:master Jun 7, 2018
@ghost ghost removed the review label Jun 7, 2018
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.

3 participants