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

Implement authentication client #4

Merged
merged 49 commits into from
Feb 6, 2025
Merged

Implement authentication client #4

merged 49 commits into from
Feb 6, 2025

Conversation

TiagoOpenCosmos
Copy link
Contributor

This MR implements an authentication client for datacosmos sdk

@TiagoOpenCosmos
Add first version of the client auth for sdk
66e0332
@TiagoOpenCosmos
Run ruff format and ruff check fix
21d0ee3
@TiagoOpenCosmos
Make some changes regarding code formatting
1efe6ed
@TiagoOpenCosmos
Add some unit tests to datacosmos client
0f1479f
@TiagoOpenCosmos
Add more unit tests
df656da
@TiagoOpenCosmos
Start fixing some of the issues reported by the pipeline
d13ce9b
@TiagoOpenCosmos
Apply black
319536e
@TiagoOpenCosmos
Revert changes in pyproject.toml
e1bee71
@TiagoOpenCosmos
Apply pydocstyle
1e40bd6
@TiagoOpenCosmos
Apply changes to workflow file
0f7da25
@TiagoOpenCosmos
Attempt to fix line too long error
03fe4d5
@TiagoOpenCosmos
another attempt to fix line too long error
311b789
@TiagoOpenCosmos
rollback changes
eb2d7c6
@TiagoOpenCosmos
Make bandit ignore B105
424f914
@TiagoOpenCosmos
Another attempt to make bandit ignore B105
dcaed8f
@TiagoOpenCosmos
Make bandit skip more unrelevant check
d9c207b
@TiagoOpenCosmos
Fix syntax main.yaml
dcfce90
@TiagoOpenCosmos
Attempt to fix line too long for client.py
57d3c7a
@TiagoOpenCosmos
Attempt to exclude files from flake8
2dc23a5
@TiagoOpenCosmos
Fix syntax in pyproject.toml
0051dc3
@TiagoOpenCosmos
Another attempt to exclude dirs from flake8
2c0ac0a
@TiagoOpenCosmos
Another attempt to exclude dirs from flake8
b3326a0
@TiagoOpenCosmos
Another attempt to exclude dirs from flake8
5da1516
@TiagoOpenCosmos
Make flake8 not deal with line too long errors
c20be93
@TiagoOpenCosmos
Attempt to make flake8 run from main.yaml taking in consideration pyp…
ee823b6 
…roject.toml file
@TiagoOpenCosmos
Another attempt to make flake8 ignore E501
bb372d9
@TiagoOpenCosmos
Attempt to make flake8 job ignore E501
34d9c14
@TiagoOpenCosmos
Attempt to make flake8 job ignore E501
3bf0038
@TiagoOpenCosmos
Attempt to make flake8 job ignore E501
e1bbed8
@TiagoOpenCosmos
Add requests package to dependencies
cbde259
pepjo
pepjo requested changes Feb 4, 2025
View reviewed changes
pepjo
pepjo requested changes Feb 5, 2025
View reviewed changes
Copy link
Member

@pepjo pepjo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be ready as soon as we provide production defaults for the config and adress the logging :)

pepjo
pepjo approved these changes Feb 6, 2025
View reviewed changes
Copy link
Member

@pepjo pepjo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@pepjo pepjo merged commit 3ecee6a into develop Feb 6, 2025
12 checks passed
@pepjo pepjo deleted the implement-authentication-client branch February 6, 2025 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pepjo pepjo pepjo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TiagoOpenCosmos @pepjo