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

Prep 4.8.0 beta00017 #795

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Prep 4.8.0 beta00017 #795

wants to merge 8 commits into from

Conversation

laimis
Copy link
Contributor

@laimis laimis commented Mar 31, 2023

This adds 4.8.0-beta00017 version mentions for the project's website and API docs. The API docs haven't been generated yet. I am waiting to ensure we can prepare and publish the release before we update and push the actual docs.

@laimis laimis requested a review from rclabo March 31, 2023 15:24
@NightOwl888 NightOwl888 added this to the 4.8.0-beta00017 milestone Oct 21, 2024
websites/site/index.md Outdated Show resolved Hide resolved
@paulirwin paulirwin self-assigned this Oct 23, 2024
@@ -0,0 +1,177 @@
---
Copy link
Contributor

Choose a reason for hiding this comment

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

The release notes need to be a separate PR from the website update. Release notes are added prior to the release vote, and the rest of these updates are added after a successful vote.

The release notes are only done this way because GitHub requires permissions to view the release notes and we send the vote out to the dev list, so many interested parties don't have those permissions. Not to mention, the draft release notes page changes URL every time it is updated.

I spotted a few other potential issues, but I will do a complete review once this is moved to a new PR.

Copy link
Contributor

Choose a reason for hiding this comment

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

IMO we should remove the website update from this PR then, given the size of the release notes. If anyone is able to help with that, help is welcome.

Copy link
Contributor

Choose a reason for hiding this comment

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

Alright. I will open a new PR. It seems simpler to remove one page from this PR than to move all of the other files to another PR along with the commit history, though. Is there a reason you suggested to do it the other way?

Copy link
Contributor

Choose a reason for hiding this comment

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

My suggestion was only because the changes to the other files were smaller and not spread across commits like the release notes are, so it seemed they would be easier to revert. Feel free to take whatever approach is easiest.


### Remaining work

See **[Current Status](xref:contributing/current-status)** for more details on the remaining work
Copy link
Contributor

Choose a reason for hiding this comment

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

Yikes, we still link here? That current status page hasn't been updated in ages. But, maybe we should keep this link and address the link to the WIKI afterward.

People keep demanding current status, but don't seem to appreciate how much effort it takes to maintain something like that. It would be nice if we had some sort of high level view somewhere that is based off of open issues and milestones, tags, etc. It might be worth investigating the GitHub features Apache supports with asf.yaml to determine whether there is anything like that where we can provide visibility without too much extra effort.

@@ -193,7 +193,7 @@ Here is an example project file for .NET 5 for testing a project named `MyCodecs
<PackageReference Include="nunit" Version="3.13.2" />
<PackageReference Include="NUnit3TestAdapter" Version="3.17.0" />
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.11.0" />
<PackageReference Include="Lucene.Net.TestFramework" Version="4.8.0-beta00016" />
<PackageReference Include="Lucene.Net.TestFramework" Version="4.8.0-beta00017" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's bump the above to .NET 8 on line 183 and <TargetFramework>net0.0</TargetFramework> on line 189.

@NightOwl888
Copy link
Contributor

Looks like we also have a conflict to resolve before this can be merged.

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.

4 participants