Skip to content

Conversation

shricodev
Copy link
Contributor

@shricodev shricodev commented Jan 17, 2023

Add docs, changelog, and authors in Development. Also fixed some minor typos and errors.

Hopefully, this fixes #2252

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 📁

Signed-off-by: Shrijal Acharya [email protected]

@shricodev
Copy link
Contributor Author

How do I fix these errors?

@AyanSinhaMahapatra
Copy link
Member

You don't need to worry about the failing test at nexB.scancode-toolkit (core_tests misc_and_scancode) as this is unrelated to your changes and will be fixed by d4fed87

@shricodev
Copy link
Contributor Author

@AyanSinhaMahapatra Sir, gentle ping. Do you happen to have any updates on this one?

Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

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

Thanks, see comments

@AyanSinhaMahapatra
Copy link
Member

@OctoPie23 can you rebase this into one commit and add a signoff so the DCO is passing properly?

@shricodev
Copy link
Contributor Author

shricodev commented Feb 18, 2023 via email

add docs, changelog and authors in Development. fixes #2252
Signed-off-by: Shrijal Acharya <[email protected]>
Signed-off-by: OctoPie23 <[email protected]>
@shricodev
Copy link
Contributor Author

shricodev commented Feb 24, 2023

@OctoPie23 can you rebase this into one commit and add a signoff so the DCO is passing properly?

@AyanSinhaMahapatra Sure!
Shall we consider merging this now?

Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks!

@AyanSinhaMahapatra AyanSinhaMahapatra merged commit 8a2c50f into aboutcode-org:develop Mar 30, 2023
@shricodev shricodev deleted the docs-edit branch March 30, 2023 14:42
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.

Update contribution guidelines to include doc, changelogs and authors

2 participants