Skip to content

clarify that stats must start with a pre-sub#1011

Open
mpadge wants to merge 16 commits into
mainfrom
pre-sub
Open

clarify that stats must start with a pre-sub#1011
mpadge wants to merge 16 commits into
mainfrom
pre-sub

Conversation

@mpadge
Copy link
Copy Markdown
Member

@mpadge mpadge commented May 20, 2026

ropensci/software-review#767


Checklist for dev guide maintainers, do not delete 😸

  • Review of the content in the initial language.
  • News item.
  • Translation of the content in other languages.
  • Review of the translations.

Comment thread softwarereview_author.Rmd Outdated
@mpadge mpadge requested a review from maelle May 21, 2026 08:50
@mpadge
Copy link
Copy Markdown
Member Author

mpadge commented May 21, 2026

@maelle Thanks for the suggestion:

should we also recommend they read that dev guide first?

I've made it a separate line now which clearly states that first, followed by the strict requirement of pre-sub. Re-review requested.

@maelle maelle force-pushed the pre-sub branch 4 times, most recently from 12c5c09 to 9f1adc3 Compare May 21, 2026 12:15
@maelle maelle requested a review from a team as a code owner May 21, 2026 12:17
@maelle maelle requested review from yabellini and removed request for a team May 21, 2026 12:17
@maelle maelle requested a review from a team as a code owner May 21, 2026 12:20
@maelle maelle requested review from fblpalmeira and removed request for a team May 21, 2026 12:20
Copy link
Copy Markdown
Member

@maelle maelle left a comment

Choose a reason for hiding this comment

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

Thank you! I added the news item and initial translations, let's wait for review.

Comment thread softwarereview_author.es.Rmd Outdated
@yabellini
Copy link
Copy Markdown
Member

Done Spanish. I already applied my changes.

@mpadge
Copy link
Copy Markdown
Member Author

mpadge commented May 22, 2026

@yabellini and @maelle : I just added another change - sorry for any inconvenience!! This one will now also need to be translated. Thank you both!

@maelle
Copy link
Copy Markdown
Member

maelle commented May 22, 2026

@mpadge this change looks unrelated to the initial PR scope, could you please

  • cherry-pick it in a brand-new PR?
  • add the news item?

Then in that PR I'll add the translations

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants