Skip to content

[BB-2320] Fix the CI checks after merge - #279

Merged
lgp171188 merged 1 commit into
masterfrom
rushal/BB-2320-problem-builder-v4-merge-fix-CI-checks
Apr 29, 2020
Merged

[BB-2320] Fix the CI checks after merge#279
lgp171188 merged 1 commit into
masterfrom
rushal/BB-2320-problem-builder-v4-merge-fix-CI-checks

Conversation

@rusrushal13

@rusrushal13 rusrushal13 commented Apr 29, 2020

Copy link
Copy Markdown
Contributor

This PR fixes the CI checks which started failing after merging the problem-builder-v4 branch to the master branch.

Test instructions:

  • verify that tests in CI reliably pass

Reviewer:

Issue: BB-2320

This will fix the failing quality checks and also the integration tests.
@rusrushal13
rusrushal13 force-pushed the rushal/BB-2320-problem-builder-v4-merge-fix-CI-checks branch from 44ba3eb to f178c68 Compare April 29, 2020 10:08
@lgp171188 lgp171188 mentioned this pull request Apr 29, 2020
@lgp171188
lgp171188 self-requested a review April 29, 2020 10:10
@lgp171188

Copy link
Copy Markdown
Contributor

@rusrushal13, the changes in this PR look good to me. Let's wait for the CI checks to pass.

@lgp171188 lgp171188 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

  • I verified that this PR fixes the failing CI checks and all the changes from the problem-builder-v4 branch are now intact in the master branch.
  • I read through the code
  • I checked for accessibility issues NA
  • Includes documentation NA

@lgp171188
lgp171188 merged commit b5500c6 into master Apr 29, 2020
@lgp171188
lgp171188 deleted the rushal/BB-2320-problem-builder-v4-merge-fix-CI-checks branch April 29, 2020 10:34
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