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.

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

Dlc purge #879

Merged
merged 4 commits into from
Mar 22, 2023
Merged

Dlc purge #879

merged 4 commits into from
Mar 22, 2023

Conversation

zbenhadi
Copy link
Contributor

No description provided.

JulesFaucherre and others added 2 commits March 20, 2023 18:28
* Delete unused code

These fields are never read from, so we can delete them.

* Delete code that doesn't do anything

This code has no effect (the endpoint variable is never used, nor the
value being written back to the struct it came from).

* Allow creating rest clients directly not from conf

Split the New into two different functions, one that takes exactly what
is needed to create the struct, and another that is able to assemble
these values from the config.

* Create a client for speaking to dl.circleci.com

- Also, add code for calling the DLC purge endpoint

* Add the command to purge DLC

---------

Co-authored-by: Abraham Tewa <[email protected]>
Co-authored-by: JulesFaucherre <[email protected]>
@zbenhadi zbenhadi requested review from a team as code owners March 21, 2023 12:17
…PE-2315/remove-api-service-from-config-compilation

PIPE-2315 - Removes API-Service from config compilation
@elliotforbes elliotforbes merged commit 8efec6d into main Mar 22, 2023
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.

5 participants