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

Update dev/from release #882

Merged
merged 24 commits into from
Oct 2, 2024
Merged

Update dev/from release #882

merged 24 commits into from
Oct 2, 2024

Conversation

MikeNeilson
Copy link
Contributor

I'm intending for this to be the last release branch based build in favor of switching to the git tag based releases we've started using in other projects. Part of that will be making the integration tests part of the merge gate (we still have 6 that are failing but it's known the work isn't done yet.)

That should help keep the number of files reduced any time I do these types of staging.

Good news is I was able to keep up with develop fairly well so these changes are mostly overriding or updating dependencies.. and fixing depedabot.

@MikeNeilson MikeNeilson added the approved-W192HQ23F0232-task4 Only valid if set by MikeNeilson, DanielO, CharlesG label Sep 26, 2024
@MikeNeilson
Copy link
Contributor Author

NOTE: there is one more change before release. Not sure if I'll do that in this style or try the new style.

We need to hide the at_properties behind a login.

@MikeNeilson
Copy link
Contributor Author

NOTE: for the TeamCity build, only the expected tests failed. Those endpoints are still in development.

@MikeNeilson
Copy link
Contributor Author

@krowvin is currently gathering if we need to give water supply contracts the same treatment. Once that's determined I'm going to call it a release.

adamkorynta
adamkorynta previously approved these changes Oct 2, 2024
Dockerfile Show resolved Hide resolved
Copy link
Collaborator

@rma-rripken rma-rripken left a comment

Choose a reason for hiding this comment

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

Can you do these two things:

  • Add unit test on the new Formats method
  • Confirm you like those test.cwbi.us urls

@MikeNeilson MikeNeilson merged commit 73f2d5e into develop Oct 2, 2024
4 of 5 checks passed
@MikeNeilson MikeNeilson deleted the update_dev/from_release branch October 2, 2024 21:06
@adamkorynta adamkorynta linked an issue Nov 7, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-W192HQ23F0232-task4 Only valid if set by MikeNeilson, DanielO, CharlesG priority:blocker priority:high
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Some New Endpoints Return Sensitive Data - Should Require Auth
4 participants