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

Mark old modules as deprecated #2482

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Mark old modules as deprecated #2482

wants to merge 1 commit into from

Conversation

blampe
Copy link
Contributor

@blampe blampe commented Oct 11, 2024

This should only be merged well after our automated upgrades have
pointed downstream providers at the new import paths.

@blampe
Copy link
Contributor Author

blampe commented Oct 11, 2024

This change is part of the following stack:

Change managed by git-spice.

Copy link

codecov bot commented Oct 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.76%. Comparing base (b9dfc43) to head (0ff3ed3).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2482   +/-   ##
=======================================
  Coverage   61.76%   61.76%           
=======================================
  Files         380      380           
  Lines       51263    51263           
=======================================
  Hits        31665    31665           
- Misses      17782    17783    +1     
+ Partials     1816     1815    -1     

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

@blampe blampe force-pushed the blampe/deprecate branch 2 times, most recently from e6a0f38 to 6708429 Compare October 15, 2024 17:29
Base automatically changed from blampe/restore to master October 15, 2024 18:32
This should only be merged well after our automated upgrades have
pointed downstream providers at the new import paths.
@iwahbe
Copy link
Member

iwahbe commented Oct 31, 2024

@blampe Can you point me to the automation that will move our providers over to the new module path?

@blampe
Copy link
Contributor Author

blampe commented Oct 31, 2024

@iwahbe pulumi/ci-mgmt#1103

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.

2 participants