hotmeteor / spectator

OpenAPI testing for PHP
MIT License
285 stars 53 forks source link

Prepare v2 #178

Closed bastien-phi closed 7 months ago

bastien-phi commented 9 months ago

Adds a composer alias for v2 and upgrade guide.

It might need to be completed

jarrodparkes commented 8 months ago

@bastien-phi aside from updating docs, what else are you and @hotmeteor desiring for v2?

hotmeteor commented 8 months ago

I think @bastien-phi has knocked out almost every single issue, which is amazing.

I'm going to chip away at some docs cleanup this morning. I don't think it needs to be enormous, but maybe just ensure it's clear what exactly this package handles, OpenAPI versioning, etc.

bastien-phi commented 8 months ago

I think it's possible to fix #140 before v2 but I did not have a lot of time recently because of client work.

I think I will get some time next week to push a couple of PRs. This would make v2 almost ready.

hotmeteor commented 8 months ago

@bastien-phi Right on

I've updated the README in this branch a little. If you have a minute, can you drop in this thread:

Basically anything you can think of that _might_be counter to what the docs did say.

I can take it from there, but you are much closer to all the nuances of the changes.

bastien-phi commented 8 months ago

@hotmeteor I think I already covered everything in the upgrade guide. I'll take a look on that but everything should be up-to-date

bastien-phi commented 8 months ago

@hotmeteor I may be wrong but Spectator::reset is not documented

bastien-phi commented 7 months ago

I think we are close from V2. @jarrodparkes can you take a look on this ? Merging this will pre-release the V2 thanks to the composer alias.

@hotmeteor I will let you hit the release button when you want !

bastien-phi commented 7 months ago

Thanks @jarrodparkes !