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

Handle extensions state #145

Merged
merged 2 commits into from
Aug 7, 2024
Merged

Conversation

jimwins
Copy link
Member

@jimwins jimwins commented Aug 7, 2024

Related to php/doc-en#3643.

@cmb69
Copy link
Member

cmb69 commented Aug 7, 2024

Great! Thank you @jimwins!

Note though that checking out to en is customary, and looking for doc-en breaks the usual expectations.

I tried the script without the doc-en changes, and it works fine.

@cmb69
Copy link
Member

cmb69 commented Aug 7, 2024

@Girgias, since you've worked on that script, you may want to have a look.

@jimwins jimwins force-pushed the handle-extensions-state branch from 9c36b11 to 50d2b26 Compare August 7, 2024 19:06
@jimwins
Copy link
Member Author

jimwins commented Aug 7, 2024

Okay, rebased without the doc-en change and fixed up my local setup.

Copy link
Member

@Girgias Girgias left a comment

Choose a reason for hiding this comment

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

Did a quick review before I'm flying, but this looks sensible

@Girgias
Copy link
Member

Girgias commented Aug 7, 2024

I'm confused why the CI is complaining about the extensions.xml file not being up to date

@cmb69
Copy link
Member

cmb69 commented Aug 7, 2024

I think the script would need to be run without the --check flag.

@jimwins
Copy link
Member Author

jimwins commented Aug 7, 2024

Because now that the script handles State, doc-en is out of date. Thus php/doc-en#3645.

@cmb69
Copy link
Member

cmb69 commented Aug 7, 2024

And doc-en is green!

@cmb69 cmb69 merged commit 279452f into php:master Aug 7, 2024
10 of 12 checks passed
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