Skip to content

Conversation

@lougeniaC64
Copy link
Contributor

Updating the metrics.yaml path specified for the logins store library in accordance with the file move that occurs in application services PR #6832.

Note (mainly to self): This PR should not be merged until the A-S PR is ready to be merged.

@travis79
Copy link
Member

As @lougeniaC64 mentioned, don't merge this until the aforementioned PR lands, at which time we can kick the CI here to ensure everything is working.

@lougeniaC64
Copy link
Contributor Author

@travis79 I got approval on the A-S PR and I'm ready to land, however I don't have the permissions to merge. Would you be able to do that?

@travis79
Copy link
Member

@travis79 I got approval on the A-S PR and I'm ready to land, however I don't have the permissions to merge. Would you be able to do that?

I sure would! Just ping me when you are ready (preferably not on GitHub as I seem to miss those some times)

@lougeniaC64 lougeniaC64 force-pushed the update-loginsStore-metrics-path branch from a4468ca to 0d51bd9 Compare July 24, 2025 18:45
@travis79 travis79 closed this Jul 25, 2025
@travis79 travis79 reopened this Jul 25, 2025
@lougeniaC64 lougeniaC64 force-pushed the update-loginsStore-metrics-path branch from 0d51bd9 to f28fe32 Compare July 25, 2025 15:31
@travis79 travis79 merged commit 0dceeec into mozilla:main Jul 25, 2025
2 checks passed
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