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 presets for metadata-extraction script #146

Merged
merged 4 commits into from
Feb 17, 2025

Conversation

eljamm
Copy link
Contributor

@eljamm eljamm commented Feb 13, 2025

  • contacts: returns a list of author e-mails to send the survey
  • overview: data to be processed for the projects overview page

Copy link

@Erethon Erethon left a comment

Choose a reason for hiding this comment

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

Tested and both -p emails|overview works as expected, LGTM. A minor nitpick, there's some extra whitespace at the end of some lines in extract_project_metadata.py that we can get rid off.

@eljamm
Copy link
Contributor Author

eljamm commented Feb 17, 2025

Nice catch! I turned off formatting for that code block to make it more readable, but that apparently turned off all formatters, not just for Python.

@fricklerhandwerk fricklerhandwerk merged commit ea85f87 into main Feb 17, 2025
@eljamm eljamm deleted the scripts-extraction-presets branch February 17, 2025 08:03
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