Skip to content

Fix test after update to postal code validation in woodwork#3544

Merged
freddyaboulton merged 2 commits intomainfrom
fix-test-after-postal-code-validation
Jun 6, 2022
Merged

Fix test after update to postal code validation in woodwork#3544
freddyaboulton merged 2 commits intomainfrom
fix-test-after-postal-code-validation

Conversation

@freddyaboulton
Copy link
Contributor

@freddyaboulton freddyaboulton commented Jun 6, 2022

Pull Request Description

Fix failure here: https://github.com/alteryx/evalml/runs/6760125434?check_suite_focus=true

Successful run here: https://github.com/alteryx/evalml/actions/runs/2449725492


After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

@codecov
Copy link

codecov bot commented Jun 6, 2022

Codecov Report

Merging #3544 (7d4d5b3) into main (ee9d333) will increase coverage by 0.1%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##            main   #3544     +/-   ##
=======================================
+ Coverage   99.7%   99.7%   +0.1%     
=======================================
  Files        335     335             
  Lines      33176   33178      +2     
=======================================
+ Hits       33048   33050      +2     
  Misses       128     128             
Impacted Files Coverage Δ
evalml/tests/component_tests/test_oversampler.py 100.0% <100.0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee9d333...7d4d5b3. Read the comment docs.

@jeremyliweishih jeremyliweishih marked this pull request as ready for review June 6, 2022 19:54
Copy link
Collaborator

@jeremyliweishih jeremyliweishih left a comment

Choose a reason for hiding this comment

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

Thank you

@freddyaboulton freddyaboulton merged commit 3251a29 into main Jun 6, 2022
@freddyaboulton freddyaboulton deleted the fix-test-after-postal-code-validation branch June 6, 2022 20:37
@freddyaboulton freddyaboulton mentioned this pull request Jun 9, 2022
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