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

Prepare for OTP 27 #507

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Conversation

paulo-ferraz-oliveira
Copy link
Contributor

@paulo-ferraz-oliveira paulo-ferraz-oliveira commented Feb 18, 2024

Description

This can/should be accepted/merged (only) when 27 is moved from release candidate status.

⚠️ Opening as draft to signal that.
⚠️ On second thought, there's an actual "issue" that might need solving, which is to use ex_doc to generate local documentation from 27 on. (this has been addressed, next to OTP, by @garazdawi, as mentioned in the linked issue)

Note: I'll try to remember to leave this here up until 27 is released, and then I'll release. If anything important comes up or is merged, I'll rebase.

Reference

@paulo-ferraz-oliveira paulo-ferraz-oliveira marked this pull request as draft February 18, 2024 14:57
@paulo-ferraz-oliveira
Copy link
Contributor Author

Locally tested on macOS. Saw HTML documentation built with ex_doc while not having it present in the installation folder.

find . -name ex_doc yields no result in the installation target.

which ex_doc doesn't find ex_doc, either before or after . ./activate.

@paulo-ferraz-oliveira
Copy link
Contributor Author

@garazdawi, feel free to give this a quick look, as I might've missed a detail somewhere 👍 Thanks.

@paulo-ferraz-oliveira
Copy link
Contributor Author

@jadeallenx, how do you wanna proceed release-wise?

Release a new version, from main, before this pr is merged, and then maybe release a 4.x-rc1 kerl after it?

Or something else?
(I'm not sure how established the API we're consuming is, at the moment).

@jadeallenx
Copy link
Collaborator

Probably should do as you suggest - it's been a minute since we did a release and it'd be best to cut one before otp 27 gets merged in

We also make sure to build more stuff in CI to cover more
functionality, and fix a bug where multiple doc targets
couldn't be bit simultaneously

Further info:
- ex_doc is not installed to the post- kerl activation
bin path (nor is it found in the target install folder)
@paulo-ferraz-oliveira
Copy link
Contributor Author

All actions are being re-run for 27.0-rc2, where applicable.

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.

Update to allow generating documentation from OTP 27 on
3 participants