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.

Sign up for GitHub

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

Jump to bottom

Separated a lot of code #11

Merged
merged 70 commits into from
Nov 25, 2024
Merged

Separated a lot of code #11

merged 70 commits into from
Nov 25, 2024

Conversation

MartinSalinas98
Copy link
Collaborator

No description provided.

@MartinSalinas98
Added get file content function
d5f585e
@MartinSalinas98
Get file content tests
82bbb43
@MartinSalinas98
Add get config date function
4ca89a6
@MartinSalinas98
Get config date function tests
47035ca
@MartinSalinas98
Simplified function to avoid reformatting
1191a53
@MartinSalinas98
Adapted tests
d536bda
@MartinSalinas98
Remove unused import
5bc7d74
@MartinSalinas98
Add parse config line function
8facea4
@MartinSalinas98
Remove unused code
dd1af70
@MartinSalinas98
Improved function docstring
6058295
@MartinSalinas98
Parse config line tests
71f665c
@MartinSalinas98
Add function make config line
8e7ce32
@MartinSalinas98
Remove old code
74cf473
@MartinSalinas98
Account for empty key
c5aff50
@MartinSalinas98
Make config line tests
d782541
@MartinSalinas98
Add config file name
78d1222
@MartinSalinas98
Show warning in parse config line function
3cf2e06
@MartinSalinas98
Simplified
116427c
@MartinSalinas98
Adapted tests
64668cf
@MartinSalinas98
Add read config function
3740351
@MartinSalinas98
Fixed test responsibilities
da20dd5
@MartinSalinas98
Created custom exception for config file invalid lines
f4f00e2
@MartinSalinas98
Invalid config line exception tests
b317f80
@MartinSalinas98
Use custom exception
a540241
@MartinSalinas98
Adapted tests
b5472bd
@MartinSalinas98
Converted to static method
5849077
@MartinSalinas98
Added test
b8531a8
@MartinSalinas98
Sorted tests
00fbc40
@MartinSalinas98
Extra test
81028c2
@MartinSalinas98
Read config general case test
c7a231a
@MartinSalinas98 MartinSalinas98 added the enhancement New feature or request label Nov 25, 2024
@MartinSalinas98 MartinSalinas98 self-assigned this Nov 25, 2024
@MartinSalinas98 MartinSalinas98 merged commit 2364974 into main Nov 25, 2024
5 checks passed
@MartinSalinas98 MartinSalinas98 deleted the code-separation branch November 25, 2024 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@MartinSalinas98 MartinSalinas98

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@MartinSalinas98