Skip to content

microprojects: add notes for the 2025 microproject list #760

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

Merged
merged 1 commit into from
Mar 24, 2025

Conversation

KarthikNayak
Copy link
Contributor

@KarthikNayak KarthikNayak commented Mar 24, 2025

From an ongoing discussion on the mailing list 1, it would be nice to add some notes to the 'Modernize a test script' microproject. Let's do that.

From an ongoing discussion on the mailing list [1], it would be nice to
add some notes to the 'Modernize a test script' microproject. Let's do
that.

[1]: https://lore.kernel.org/all/CAPig+cRm+sc+Rk-4SuQ5CrPeZLG2Nzz9B7+6OZxCq7tV5mzmBA@mail.gmail.com/
@KarthikNayak KarthikNayak force-pushed the kn-update-micro-projects branch from 7ac0419 to ea4a0d9 Compare March 24, 2025 14:21
@chriscool chriscool merged commit fa9fe83 into git:master Mar 24, 2025
@chriscool
Copy link
Collaborator

Thanks @KarthikNayak !

Copy link
Member

@gitster gitster left a comment

Choose a reason for hiding this comment

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

This item is about "Modernize a test script", so your Notes #1 (only work on t/t????-*.sh) is trivially correct, #2 makes sense, BUT listing #3 together with them does not make any sense. The quoted message from Eric talks many things and only one bullet item among 17 bullets is about "test -[def]".

@KarthikNayak
Copy link
Contributor Author

This item is about "Modernize a test script", so your Notes #1 (only work on t/t????-*.sh) is trivially correct, #2 makes sense, BUT listing #3 together with them does not make any sense. The quoted message from Eric talks many things and only one bullet item among 17 bullets is about "test -[def]".

Okay. I get what you're saying, it would make sense to clarify which point we're referring to. I thought that inference could be drawn, but better to make it explicit.

@KarthikNayak
Copy link
Contributor Author

@chriscool @gitster I have raised #761 as a follow up

@chriscool
Copy link
Collaborator

Thanks @KarthikNayak and @gitster ! I have just merged #761.

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