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

changelog: start listing changes since 0.12 #134

Merged
merged 1 commit into from
Jan 30, 2023
Merged

changelog: start listing changes since 0.12 #134

merged 1 commit into from
Jan 30, 2023

Conversation

jnikula
Copy link
Owner

@jnikula jnikula commented Jan 30, 2023

Start listing changes since 0.12, with hopes of gearing towards a 0.13 release in the near future. There's been a lot of changes since 0.12, maybe more than in any other release, and I'd like to get it out sooner rather than later. (Release early, release often, and all that.)

@BrunoMSantos How does the hat-tip to your employer look like?

I think the main blockers for 0.13 are documenting the C++ support, as well as adding any C++ features that you think we absolutely need before making a release. Not to put any pressure on you. ;)

I'm also fine with downgrading "Support for documenting C++" to "Initial support for documenting C++", depending on the level of support and/or boasting.

@BrunoMSantos
Copy link
Collaborator

Start listing changes since 0.12, with hopes of gearing towards a 0.13 release in the near future. There's been a lot of changes since 0.12, maybe more than in any other release, and I'd like to get it out sooner rather than later. (Release early, release often, and all that.)

Indeed!

@BrunoMSantos How does the hat-tip to your employer look like?

Think it's good given what I discussed with them.

I think the main blockers for 0.13 are documenting the C++ support, as well as adding any C++ features that you think we absolutely need before making a release. Not to put any pressure on you. ;)

I'm also fine with downgrading "Support for documenting C++" to "Initial support for documenting C++", depending on the level of support and/or boasting.

Right, the critical piece missing is namespaces I think, but I haven't made it a priority. I'm trying to improve a few things in the parser 1st. How do you feel about waiting until this next weekend? If it's not going fast enough, we release earlier, otherwise we may attempt a more complete release (C++ wise).

I'd also be ok with documenting what we have and leave a loud warning that it is experimental with a list of stuff we know is missing. Just the other day I remembered a few more missing things in fact, though nothing nearly as critical as namespaces.

CHANGELOG.rst Outdated
Added
~~~~~

* Support for documenting C++ (contributions courtesy of `Critical Software`_)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure if you intend to merge this PR as it is for us to add things as we go along (not a bad idea by the way, and do so from now on for upcoming releases).

Anyway, if that's the case, I would only ask you to temper this sentence. Call it 'early support for' or 'cpp domain enablement' for now.

Copy link
Owner Author

Choose a reason for hiding this comment

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

I think I'll call it "early" or something for now, and we can adjust it later before the release, depending on what we have.

It would be great to update the changelog with the changes as they come instead of right before release, but nobody ever likes to write changelogs. :)

@jnikula
Copy link
Owner Author

jnikula commented Jan 30, 2023

Right, the critical piece missing is namespaces I think, but I haven't made it a priority. I'm trying to improve a few things in the parser 1st. How do you feel about waiting until this next weekend? If it's not going fast enough, we release earlier, otherwise we may attempt a more complete release (C++ wise).

I'm not that much in a rush, I'm leaning towards releasing something that's usable for documenting C++. Doesn't have to be complete, but I also don't want folks to try it out and think it's unusable. It's fine if people think "this doesn't cover the obscure feature I need, bummer" but less so if they go "can't do anything, what a waste of time to even try".

I'm also depending on you to define what "usable" means here. :)

@BrunoMSantos
Copy link
Collaborator

Right, the critical piece missing is namespaces I think, but I haven't made it a priority. I'm trying to improve a few things in the parser 1st. How do you feel about waiting until this next weekend? If it's not going fast enough, we release earlier, otherwise we may attempt a more complete release (C++ wise).

I'm not that much in a rush, I'm leaning towards releasing something that's usable for documenting C++. Doesn't have to be complete, but I also don't want folks to try it out and think it's unusable. It's fine if people think "this doesn't cover the obscure feature I need, bummer" but less so if they go "can't do anything, what a waste of time to even try".

I'm also depending on you to define what "usable" means here. :)

Right, then I'd say we need namespaces for sure. It's an almost universal pattern to put everything in namespace blocks, and we don't recurse into those at the moment.

@jnikula
Copy link
Owner Author

jnikula commented Jan 31, 2023

@BrunoMSantos Also, let me know if there's anything I could do to help. Like, maybe I could pick up what you had for documentation and rebase and polish that? Or any blockers?

@BrunoMSantos
Copy link
Collaborator

Thanks, I'll push the documentation part in case you want to have a look then. I don't think I changed it much since the version you saw.

Right now I'm trying to change the docstring API so that it takes a cursor (not Clang's cursor exactly, but a more documentation-centric one). The clean-up is promising, but the diffs are huge and it just takes time. Not sure there's a huge benefit in doing this before or after the namespace thing, but I was nerd snipped into it and that's what I'm going with 😅

@jnikula
Copy link
Owner Author

jnikula commented Jan 31, 2023

Not sure there's a huge benefit in doing this before or after the namespace thing, but I was nerd snipped into it and that's what I'm going with sweat_smile

I feel you! xD

@BrunoMSantos
Copy link
Collaborator

Thanks, I'll push the documentation part in case you want to have a look then. I don't think I changed it much since the version you saw.

Pushed here: https://github.com/BrunoMSantos/hawkmoth/tree/cpp-docs

@jnikula jnikula deleted the changelog branch February 26, 2023 14:08
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.

None yet

2 participants