Skip to content

Referenced_filenames should be returned by API function - #2632

Merged
pombredanne merged 2 commits into
aboutcode-org:developfrom
akugarg:update_api
Aug 12, 2021
Merged

Referenced_filenames should be returned by API function#2632
pombredanne merged 2 commits into
aboutcode-org:developfrom
akugarg:update_api

Conversation

@akugarg

@akugarg akugarg commented Aug 10, 2021

Copy link
Copy Markdown
Contributor

Signed-off-by: akugarg akanksha.garg2k@gmail.com

Check #2616 (comment) for more details

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 📁

Comment thread src/licensedcode/models.py Outdated
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
@akugarg
akugarg force-pushed the update_api branch 8 times, most recently from 8977b09 to 3bbaf27 Compare August 11, 2021 13:42
@akugarg

akugarg commented Aug 11, 2021

Copy link
Copy Markdown
Contributor Author

@AyanSinhaMahapatra All green!
@pombredanne plz have a look!

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! Looks ready.

Comment thread tests/summarycode/data/full_summary/summary_key_files.expected.json Outdated
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Perfect!
LGTM.... Please also add a CHANGELOG.rst entry for this and the feature of follwing these references (this can be in another PR)

@pombredanne
pombredanne merged commit daeb709 into aboutcode-org:develop Aug 12, 2021
@akugarg
akugarg deleted the update_api branch August 12, 2021 17:47
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