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

Apply fnmatch / re match to match file name #83

Merged
merged 10 commits into from
Jul 15, 2022
Merged

Apply fnmatch / re match to match file name #83

merged 10 commits into from
Jul 15, 2022

Conversation

bjk7119
Copy link
Contributor

@bjk7119 bjk7119 commented Jul 15, 2022

Description

  • Changed logic to find and remove files excluded from yaml
    • AS-IS: Find list of files to exclude from yaml in files without license/copyright and remove them later.
    • TO-BE: If you find a file to be excluded from yaml in a file without license/copyright, remove it immediately.
  • OSS Package file is excluded from the list of files without license/copyright
  • The name of the yaml file to parse:
    • AS-IS: oss-pkg-info.yaml
    • TO-BE: oss-pkg-info*.yaml
  • How to match against files written to yaml:
    • AS-IS: File name pattern match using fnmath
    • TO-BE: Match the file name pattern using fnmath, and if it does not match, match regular expression
      Reason: When only the folder name is written (ex. In case of temp, to match all temp/*)
  • Fixed bug where oss-pkg-info*.yaml file could not be found in OSS Package file list

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation update
  • Refactoring, Maintenance
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

@bjk7119 bjk7119 added the chore [PR/Issue] Refactoring, maintenance the code label Jul 15, 2022
@bjk7119 bjk7119 self-assigned this Jul 15, 2022
@bjk7119 bjk7119 changed the title Change the logic to exclude files Apply fnmatch / re match to match file name Jul 15, 2022
@bjk7119 bjk7119 merged commit aec5bf1 into main Jul 15, 2022
@bjk7119 bjk7119 deleted the fnmatch_new branch July 18, 2022 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore [PR/Issue] Refactoring, maintenance the code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants