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

[issue-261] delete duplicated files #317

Merged
merged 2 commits into from
Nov 28, 2022

Conversation

meretp
Copy link
Collaborator

@meretp meretp commented Nov 25, 2022

fixes #261

Signed-off-by: Meret Behrens meret.behrens@tngtech.com

Copy link
Collaborator

@nicoweidner nicoweidner left a comment

Choose a reason for hiding this comment

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

We should rephrase slightly in the readme, since we have to expect users will be using a built distribution, not a source distribution

README.md Outdated
@@ -51,7 +51,7 @@ This is the result of an initial GSoC contribution by @[ah450](https://github.co

1. **PARSER** (for parsing any format):
* Use `pyspdxtools_parser --file <filename>` where `<filename>` is the location of the file.
Try running: `pyspdxtools_parser --file data/SPDXRdfExample.rdf`.
Try running: `pyspdxtools_parser --file tests/data/formats/SPDXRdfExample.rdf`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

The example files in test/data/formats/ are not included in built distributions, only in source distributions (the same is true for the example files in data/, for that matter). We should rephrase these comments to something like "If you are using a source distribution, try running..."

… in README

Signed-off-by: Meret Behrens <meret.behrens@tngtech.com>
Signed-off-by: Meret Behrens <meret.behrens@tngtech.com>
@meretp meretp force-pushed the delete_duplicated_example_files branch from 99c4cc1 to c210c56 Compare November 28, 2022 07:19
@meretp meretp merged commit d99438b into spdx:main Nov 28, 2022
@meretp meretp deleted the delete_duplicated_example_files branch November 28, 2022 08:39
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.

delete duplicated example files
2 participants