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

Add 3.7.1 Release Notes #2583

Merged
merged 2 commits into from Feb 1, 2024
Merged

Add 3.7.1 Release Notes #2583

merged 2 commits into from Feb 1, 2024

Conversation

Griffin-Sullivan
Copy link
Contributor

Please cherry-pick my commits into:

  • Foreman 3.8/Katello 4.10
  • Foreman 3.7/Katello 4.9 (planned Satellite 6.14)
  • Foreman 3.6/Katello 4.8
  • Foreman 3.5/Katello 4.7 (Satellite 6.13; orcharhino 6.6)
  • Foreman 3.4/Katello 4.6 (EL8 only)
  • Foreman 3.3/Katello 4.5 on EL7 & EL8 (Satellite 6.12 on EL8 only; orcharhino 6.4/6.5 on EL8 only)
  • Foreman 3.2/Katello 4.4 on EL7 & EL8
  • Foreman 3.1/Katello 4.3 on EL7 & EL8 (Satellite 6.11 EL7/8; orcharhino 6.3 on EL7/8)
  • We do not accept PRs for Foreman older than 3.1.

@Griffin-Sullivan
Copy link
Contributor Author

Let me know if I messed up the contributors section. It looked like it hadn't been updated since 3.7.0-rc1. I ran it from 3.6.0 to 3.7.1. Is that correct?

@@ -1,5 +1,4 @@
We'd like to thank the following people who contributed to the Foreman {{page.version}} release:

Adam Ruzicka, Alex Fisher, Amit Upadhye, Anthony Somerset, Bastian Schmidt, Bernhard Suttner, Chris Roberts, Christian Ruppert, Dirk Götz, Dirk Heinrichs, Dyrkon, Eric D. Helms, Evgeni Golov, Ewoud Kohl van Wijngaarden, Frank Adaemmer, Ian Ballou, Jeremy Lenz, Jonas Trüstedt, Kamil Szubrycht, Karolina Malyjurkova, Leos Stejskal, Lior Keren, Lucy Fu, Lukáš Zapletal, Manuel Laug, Marcel Kühlhorn, Marek Hulán, Maria Agaphontzev, Matt Darcy, Matěj Mudra, Nofar Alfassi, Oleh Fedorenko, Partha Aji, Pat Riehecky, Pavel Moravec, Peter Koprda, Quirin Pamp, Ron Lavi, Samir Jha, Sayan Das, Shim Shtein, William Clark, benjamin-robertson, kobybr, timdeluxe, Štefan Németh

Adam Hosek, Adam Ruzicka, Alex Fisher, Alexey Masolov, Andrew Teixeira, Antonin Dvorak, Archana Kumari, Areyus, Bastian Schmidt, Bernhard Suttner, Damon Clinkscales, Daniel Alley, Dirk Götz, Eric Cook, Eric D. Helms, Erik Berg, Evgeni Golov, Ewoud Kohl van Wijngaarden, Girija Soni, Gordon Bleux, Griffin Sullivan, Ian Ballou, Jan Klare, Jerome Charaoui, Joshua Hoblitt, Leos Stejskal, Lior Keren, Lucy Fu, Marek Hulán, Maria Agaphontzev, Markus Bucher, Maximilian Kolb, Nagoor Shaik, Nofar Alfassi, Oleh Fedorenko, Pat Riehecky, Peter Bittner, Peter Koprda, Romain Tartière, Ron Lavi, Samir Jha, Sebastian Bublitz, Tim Meusel, Trey Dockendorf, Vijaykumar Sawant, William Clark, chr1s692, jc
Copy link
Member

Choose a reason for hiding this comment

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

Somehow I no longer see Anthony Somerset here.

Recently @wbclark pointed out that you can put everyone on their own line and in the rendered document it ends up as a single sentence. I think the best approach is to replace it first, commit that and then replace the whole set. The line by line comparison should make it very easy to see.

So use sed -i 's/, /,\n/g' guides/doc-Release_Notes/topics/foreman-contributors.adoc.

I've also opened theforeman/theforeman.org#2112 to automate the replacement of names.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok I placed them on their own lines. Was I supposed to add Anthony back?

Copy link
Member

Choose a reason for hiding this comment

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

I don't have enough context to explicitly know of Anthony was, but I would assume that anyone present as a contributor in 3.7.0 should also be a contributor in 3.7.1

Copy link
Contributor

@mjivraja mjivraja left a comment

Choose a reason for hiding this comment

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

LGTM style-wise and the links are tested.

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

What I meant was first create a commit that changes the list of contributors to multi line. Pretty much the output of sed -i 's/, /,\n/g' guides/doc-Release_Notes/topics/foreman-contributors.adoc except that also changes another unrelated line. Then update the list of contributors.

If you do that (and I did to show it) then you get ekohl@678804d where you can easily see which contributors are no longer listed.

That's quite a large list and I wonder what happened. Was it wrong in the first place, or is there an issue now?

Perhaps it's better to move forward with the release and only do the release notes now, leave the contributors to a separate step.

@pr-processor pr-processor bot added the Waiting on contributor Requires an action from the author label Dec 1, 2023
@Griffin-Sullivan
Copy link
Contributor Author

I just rebased this. Not sure what the next step should be @ekohl

@ekohl ekohl merged commit 19eee13 into theforeman:3.7 Feb 1, 2024
8 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.

None yet

4 participants