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

Deprecate the deprecator #494

Merged
merged 8 commits into from
Nov 13, 2023
Merged

Deprecate the deprecator #494

merged 8 commits into from
Nov 13, 2023

Conversation

zigaLuksic
Copy link
Collaborator

The deprecator from typing_extensions has additional benefits of IDEs picking it up

image

I wanted to deprecate our own deprecators, but then users would get spammed with stuff that they have no control over (since it would happen on import. So I want to wait with that (or just remove them after a while)

mlubej and others added 4 commits November 7, 2023 16:16
* update actions and gitlab CI

* update project path

* use trigger API to build internal docker instead

* test trigger on branch push

* fix github action

* test why trigger workflow not starting

* fix branch name

* remove step in trigger job

* remove test branch

* move mirror to a separate workflow, update pipeline running rules

* Revert "move mirror to a separate workflow, update pipeline running rules"

This reverts commit 92b4b04.

---------

Co-authored-by: Matic Lubej <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Nov 13, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7fbdcd7) 91.95% compared to head (6f8cbb4) 91.95%.
Report is 2 commits behind head on master.

❗ Current head 6f8cbb4 differs from pull request most recent head d481afc. Consider uploading reports for the commit d481afc to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #494   +/-   ##
=======================================
  Coverage   91.95%   91.95%           
=======================================
  Files          55       55           
  Lines        4800     4800           
=======================================
  Hits         4414     4414           
  Misses        386      386           
Files Coverage Δ
sentinelhub/api/byoc.py 94.54% <100.00%> (ø)
sentinelhub/api/utils.py 96.00% <100.00%> (ø)
sentinelhub/geometry.py 85.92% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@mlubej mlubej left a comment

Choose a reason for hiding this comment

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

You plan to do a separate MR for preparing the release, right?

@zigaLuksic
Copy link
Collaborator Author

You plan to do a separate MR for preparing the release, right?

yes

@zigaLuksic zigaLuksic merged commit e33b58d into master Nov 13, 2023
7 checks passed
@zigaLuksic zigaLuksic deleted the deprecate-the-deprecator branch November 13, 2023 10:32
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.

3 participants