Skip to content
This repository was archived by the owner on Mar 13, 2024. It is now read-only.

Changed the ignore functionality so individual lines in files can be ignored#77

Merged
evvaaaa merged 6 commits into
mainfrom
incorrect_organization_replacement_in_general_README
Apr 17, 2023
Merged

Changed the ignore functionality so individual lines in files can be ignored#77
evvaaaa merged 6 commits into
mainfrom
incorrect_organization_replacement_in_general_README

Conversation

@evvaaaa
Copy link
Copy Markdown
Contributor

@evvaaaa evvaaaa commented Mar 16, 2023

closes #75

IGNORE_FILES is now a dictionary, where keys are string names of files to be ignored, and values are of type List[int]. If the value is an empty list then the file will be ignored completely and not replaced. If the value contains ints then those lines will be ignored in that file, and the rest of the file will run through the same replacement as before. int line numbers start at 1 and not 0 (the first line in a file is 1).

Tested with an exisiting run where the specified line in the README (from issue #75) was correctly ignored with the rest of the file swapped.

@evvaaaa evvaaaa requested a review from coretl March 16, 2023 14:45
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 16, 2023

Codecov Report

Merging #77 (de784b4) into main (67dfa74) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #77   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines          199       223   +24     
=========================================
+ Hits           199       223   +24     
Impacted Files Coverage Δ
src/python3_pip_skeleton/__main__.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@evvaaaa evvaaaa force-pushed the incorrect_organization_replacement_in_general_README branch 2 times, most recently from 303dd18 to b446bd5 Compare March 16, 2023 15:19
Comment thread src/python3_pip_skeleton/__main__.py Outdated
@evvaaaa evvaaaa force-pushed the incorrect_organization_replacement_in_general_README branch from b446bd5 to 12179a8 Compare March 17, 2023 09:36
@evvaaaa evvaaaa self-assigned this Mar 17, 2023
Comment thread src/python3_pip_skeleton/__main__.py Outdated
@evvaaaa evvaaaa force-pushed the incorrect_organization_replacement_in_general_README branch from 78b848d to 7d0828c Compare April 12, 2023 14:44
@evvaaaa evvaaaa force-pushed the incorrect_organization_replacement_in_general_README branch from 7d0828c to 14a16cb Compare April 12, 2023 14:49
@evvaaaa evvaaaa requested a review from coretl April 13, 2023 07:44
Comment thread src/python3_pip_skeleton/__main__.py Outdated
Comment thread src/python3_pip_skeleton/__main__.py Outdated
@evvaaaa evvaaaa force-pushed the incorrect_organization_replacement_in_general_README branch from 3310e9d to c90478e Compare April 14, 2023 09:21
Comment thread src/python3_pip_skeleton/__main__.py Outdated
@evvaaaa evvaaaa force-pushed the incorrect_organization_replacement_in_general_README branch 2 times, most recently from 628e714 to c256058 Compare April 14, 2023 10:25
@evvaaaa evvaaaa force-pushed the incorrect_organization_replacement_in_general_README branch from c256058 to de784b4 Compare April 14, 2023 10:30
@evvaaaa evvaaaa requested a review from coretl April 17, 2023 07:44
@evvaaaa evvaaaa merged commit 7785906 into main Apr 17, 2023
@evvaaaa evvaaaa deleted the incorrect_organization_replacement_in_general_README branch April 17, 2023 13:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect Organization replacement in generated README

5 participants