Skip to content

Update conf files location#1358

Merged
omri374 merged 18 commits into
mainfrom
omri/conf_addition_test
Apr 18, 2024
Merged

Update conf files location#1358
omri374 merged 18 commits into
mainfrom
omri/conf_addition_test

Conversation

@omri374

@omri374 omri374 commented Apr 15, 2024

Copy link
Copy Markdown
Collaborator

Change Description

This PR changes the default location of the analyzer's conf files to be inside the package (i.e. inside presidio_analyzer) so that it gets added to the wheel file that's uploaded to PyPI.

Checklist

  • I have reviewed the contribution guidelines
  • I have signed the CLA (if required)
  • My code includes unit tests
  • All unit tests and lint checks pass locally
  • My PR contains documentation updates / additions if required

@omri374

omri374 commented Apr 15, 2024

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@omri374 omri374 changed the title TEST: DO NOT PUSH TO MAIN Update conf files location Apr 15, 2024
@omri374 omri374 requested a review from roeybc April 15, 2024 20:08
Comment thread presidio-analyzer/MANIFEST.in Outdated
Comment thread presidio-analyzer/presidio_analyzer/conf/default.yaml Outdated
@omri374 omri374 marked this pull request as ready for review April 15, 2024 20:18
Comment thread presidio-analyzer/Dockerfile.windows Outdated
SharonHart
SharonHart previously approved these changes Apr 16, 2024
@omri374 omri374 merged commit f29e112 into main Apr 18, 2024
@omri374 omri374 deleted the omri/conf_addition_test branch April 18, 2024 07:42
areyesfalcon added a commit to areyesfalcon/presidio that referenced this pull request Apr 18, 2024
prokopidis pushed a commit to prokopidis/presidio that referenced this pull request Jun 23, 2026
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