Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

added link_batch and removed an unused function #6

Merged
merged 2 commits into from
Dec 1, 2022
Merged

Conversation

ketakipai
Copy link
Contributor

@ketakipai ketakipai requested review from guyd and koryf December 1, 2022 16:32
@ketakipai ketakipai merged commit f3aa7ef into dev Dec 1, 2022
@ketakipai ketakipai deleted the minor-corrections branch December 1, 2022 16:37
ketakipai added a commit that referenced this pull request Dec 6, 2022
* added all files

* updated README

* Deid 1284 improve robustness of pii check on off flags (#3)

* fixed bugs with the PII_CHECK flags

* removed local hook

* added suggestion by Guy

* added Guys suggestion and removed local hook

* DEID-1311-update-git-pre-commit-hook-documentation (#4)

* updated README

* did the suggested changes

* Update the way API KEY is read (#2)

* passing path of env file as variable

* remove the unnecessary line from setup.py

* bug fix and updated script to get api key from env file

* updated README

* removed pathlib2 dependency and added stuff to check for relative path

* updated README

* added validation to check if API_KEY is missing

* updated README

* added all dependencies in setup.py, removed from config

* removing unnecessary dependecies from pre-commit-hooks

* updated dependencies

* removed unnecessary fields mentioned in README, tested and confirmed that we need the rest

* updated README

* updated README

* updated README

* updated pre-commit-hooks

* removing args from pre-commit-hooks

* removing entry from pre-commit-config

* updated README to resolve conflicts

* Make compatible with beta2 (#5)

* test

* updated fields to be compatible with beta 2

* removing line added  for debugging

* updated README to contain latest rev

* update README to have new repo

* added link_batch and removed an unused function (#6)

* added link_batch and removed an unused function

* removed test file
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants