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

[tests-only] Bump commit ids for tests #881

Closed
wants to merge 2 commits into from

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented Nov 18, 2020

  1. Bump core commit id for API tests - this will include any test code changes in:
    [tests-only] Fix test steps for when REPLACE_USERNAMES is in effect core#38112 [tests-only] Fix test steps for when REPLACE_USERNAMES is in effect

  2. Bump phoenix commit id for UI tests - this will include any test code changes in:
    [tests-only] added issue tag in imageMediaViewer test web#4299 [tests-only] added issue tag in imageMediaViewer test
    Fix dynamic client registration web#4306 Fix dynamic client registration
    Move actions to sidebar web#4255 Move actions to sidebar
    Version bump for release 0.25.0 web#4313 Version bump for release 0.25.0
    Note: probably we will need to also update the actual phoenix code being run - see Update phoenix 0.25.0 #868

@phil-davis phil-davis self-assigned this Nov 18, 2020
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@phil-davis
Copy link
Contributor Author

core API tests pass. phoenix UI tests are failing. I will make a PR for just the core commit id bump...

@phil-davis
Copy link
Contributor Author

See #883

@phil-davis phil-davis closed this Nov 18, 2020
@phil-davis phil-davis deleted the bump-commit-ids-for-tests branch November 18, 2020 10:03
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.

1 participant