Dark Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Mention that in Fedora you can now install scancode from Fedora repo#3824

Merged
AyanSinhaMahapatra merged 3 commits intoaboutcode-org:developfrom
xsuchy:patch-1
Jun 26, 2024
Merged

Mention that in Fedora you can now install scancode from Fedora repo#3824
AyanSinhaMahapatra merged 3 commits intoaboutcode-org:developfrom
xsuchy:patch-1

Conversation

Copy link
Contributor

xsuchy commented Jun 24, 2024 *
edited by AyanSinhaMahapatra
Loading

See https://bodhi.fedoraproject.org/updates/?packages=scancode-toolkit

Fixes: #3095

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled and links the original issue above
  • Tests pass -- look for a green checkbox a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts
  • Updated documentation pages (if applicable)
  • Updated CHANGELOG.rst (if applicable)

AyanSinhaMahapatra reacted with hooray emoji AyanSinhaMahapatra, pombredanne, and keshav-space reacted with heart emoji
xsuchy force-pushed the patch-1 branch from 0c4bc4f to 5fbdb81 Compare June 24, 2024 12:39
Copy link
Member

AyanSinhaMahapatra commented Jun 24, 2024

Thanks a lot @xsuchy!
Could you also mention this in the installation docs at https://github.com/nexB/scancode-toolkit/blob/develop/docs/source/getting-started/install.rst?plain=1 so this shows up at https://scancode-toolkit.readthedocs.io/en/latest/getting-started/install.html

This would be a new section and reference here at https://github.com/nexB/scancode-toolkit/blob/develop/docs/source/getting-started/install.rst?plain=1#L414 and a reference at https://github.com/nexB/scancode-toolkit/blob/develop/docs/source/getting-started/install.rst?plain=1#L36, both like the pip install sections above.

xsuchy reacted with thumbs up emoji

Copy link
Member

pombredanne commented Jun 25, 2024

@xsuchy you rock

Copy link
Contributor Author

xsuchy commented Jun 25, 2024

@xsuchy you rock

Nah. I did just a fraction of the work. Credits go to Eclipseo.

xsuchy force-pushed the patch-1 branch from 5fbdb81 to 36ef304 Compare June 25, 2024 11:53
Copy link
Contributor Author

xsuchy commented Jun 25, 2024

Thanks a lot @xsuchy! Could you also mention this in the installation docs

Updated.

AyanSinhaMahapatra reacted with heart emoji

AyanSinhaMahapatra approved these changes Jun 26, 2024
Copy link
Member

AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

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

@xsuchy Thanks!
We need just these tiny changes to highlight these correctly in the docs, and this is ready to merge otherwise.

xsuchy reacted with thumbs up emoji
xsuchy and others added 2 commits June 26, 2024 15:11
Co-authored-by: Ayan Sinha Mahapatra
Co-authored-by: Ayan Sinha Mahapatra
Copy link
Member

AyanSinhaMahapatra commented Jun 26, 2024

Thanks! Merging this.

AyanSinhaMahapatra merged commit 9a6354d into aboutcode-org:develop Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

AyanSinhaMahapatra AyanSinhaMahapatra approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Build RPMs for scancode

3 participants