Skip to content

chore(report) : Add community bonding report #211

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 3 commits into from
Jun 7, 2024

Conversation

rajuljha
Copy link
Contributor

@rajuljha rajuljha commented Jun 5, 2024

Project

Community Bonding report for @rajuljha for GSoC' 24

Changes

Add community bonding report and add personal as well as project details.

@rajuljha rajuljha requested a review from GMishx June 6, 2024 07:47
Copy link
Member

@avinal avinal left a comment

Choose a reason for hiding this comment

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

Slight change, rest looks good to me.

Comment on lines 27 to 36
To be able to easily and continuously scan packages with
fossology checks in CI pipelines, a docker image
(fossology/fossology:scanner) capable of running license checks (using
nomos or ojo) and keyword and copyright scans is available.
This project aims to improve the CI Scanner Image in
various aspects and numerous quality of life improvements, like
highlighting the exact location of violation, ability to customize the
keywords used by the scanner, and improving user experience – allow
whitelisting from a custom location and ability to download and scan
dependencies
Copy link
Member

Choose a reason for hiding this comment

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

Please simplify this. Either write it in bullet points or separate the paragraphs. Try active voice.

Copy link
Member

Choose a reason for hiding this comment

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

Okay for community bonding, but for next updates, please try to include more from the discussion, like a few lines about the solution that was discussed/accepted.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have updated the project info as well as provided more meaningful community bonding report.

Copy link
Member

@avinal avinal left a comment

Choose a reason for hiding this comment

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

Looks good to me

@GMishx
Copy link
Member

GMishx commented Jun 7, 2024

@rajuljha please update the commit message "Update report date" according to the contributing guidelines.

Hint: you can modify the message when you do interactive rebase.

@rajuljha rajuljha force-pushed the chore/report/community-bonding branch from bf20acc to 990fa2a Compare June 7, 2024 08:26
@rajuljha
Copy link
Contributor Author

rajuljha commented Jun 7, 2024

@rajuljha please update the commit message "Update report date" according to the contributing guidelines.

Hint: you can modify the message when you do interactive rebase.

Thanks for the tip! Please take a look if it is fine now?

@GMishx GMishx enabled auto-merge June 7, 2024 08:42
@GMishx GMishx merged commit dd56bd7 into fossology:main Jun 7, 2024
2 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.

3 participants