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

Schema v11 Update & release [1/3][🎃 hacktober] #169

Merged
merged 17 commits into from
Oct 6, 2021

Conversation

mallardduck
Copy link
Contributor

This is the second schema upgrade and next release PR.
With this set of PRs I'm going to be making sure to get the docs along the way too.
That way #168 will be the only errant one with only docs fixes to account for tagging a release.

the previous keys no longer are used. the "schema:" prefix is used now instead

(cherry picked from commit 19d1515)
Rather than updating all of our logic to parse things differently - this simply corrects all the prefixed data to expanded form.
From there, the logic that pulls things together works fine again. This enables the generator to be updated sooner than later.
Given that the generator is run rarely a more efficient strategy could be created via building a new generator logic at any time without affecting end-user versions.

(cherry picked from commit ef09856)
* docs-schema-v10:
  Add helpful table...
  update changelog for accepted PRs in schema V10 based release
...no sense in releasing with the unpatched v11 that had errors.
@mallardduck mallardduck changed the title Schema v11 Update & v5 release [1/?][🎃 hacktober] Schema v11 Update & v5 release [1/3][🎃 hacktober] Oct 5, 2021
Fix merge conflicts and update changelog
rather than fixing it lets remove it to match the rest of teh generator code
@mallardduck mallardduck changed the title Schema v11 Update & v5 release [1/3][🎃 hacktober] Schema v11 Update & release [1/3][🎃 hacktober] Oct 5, 2021
@mallardduck
Copy link
Contributor Author

Updated to remove the table from readme file and got changelog to match previous PR/release.
(all subsequent PRs are updated with the same as well)

@Gummibeer Gummibeer added the Hacktoberfest https://hacktoberfest.digitalocean.com label Oct 5, 2021
@mallardduck mallardduck requested a review from Gummibeer October 5, 2021 17:23
@mallardduck
Copy link
Contributor Author

Using the awesome php-semver-checker tool I am more easily able to compare the changes than on GitHub's diff page. This way I can point the tool to only analyze the src folder too.

While it does suggest this be a "MAJOR" release, this due to all sorts of new added methods. Which it considers as major changes by default rather than a "MINOR" change. That said, it no longer includes any removals!

So the subsequent PRs will need to be tweaked...but I figure I'll adjust those once this one is merged in and those are synced with master branch.

@Gummibeer
Copy link
Collaborator

Awesome! 🎉
Amazing work like always! 🚀
Quick scroll through hasn't shown any big red blocks - so I trust you. 😂
I also agree that adding methods isn't breaking - so we can go ahead. 🥳

One of the most productive and serious PR series for https://hacktoberfest.digitalocean.com/ I've ever seen! 🤯

OUTSTANDING

@Gummibeer Gummibeer merged commit ed00fc4 into spatie:master Oct 6, 2021
@Gummibeer
Copy link
Collaborator

https://github.com/spatie/schema-org/releases/tag/3.6.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hacktoberfest https://hacktoberfest.digitalocean.com hacktoberfest-accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants