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

Fix dart deprecation warning #263

Merged

Conversation

scpeters
Copy link
Member

🦟 Bug fix

Fix for dart deprecation warning copied from #262.

Summary

Keep using the old method for dart 6.9 and earlier, but use the new method for 6.10+.

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

Copied from gazebosim#262.

Signed-off-by: Steve Peters <[email protected]>
@scpeters scpeters requested a review from chapulina June 17, 2021 19:12
@scpeters scpeters requested a review from mxgrey as a code owner June 17, 2021 19:12
@github-actions github-actions bot added the 🏰 citadel Ignition Citadel label Jun 17, 2021
@scpeters scpeters mentioned this pull request Jun 17, 2021
@codecov
Copy link

codecov bot commented Jun 17, 2021

Codecov Report

Merging #263 (c0a1dde) into ign-physics2 (1c04c60) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##           ign-physics2     #263   +/-   ##
=============================================
  Coverage         82.94%   82.94%           
=============================================
  Files               106      106           
  Lines              4034     4034           
=============================================
  Hits               3346     3346           
  Misses              688      688           

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 1c04c60...c0a1dde. Read the comment docs.

@scpeters scpeters merged commit 817430b into gazebosim:ign-physics2 Jun 17, 2021
@scpeters scpeters deleted the scpeters/fix_dart_deprecation branch June 17, 2021 21:22
chapulina added a commit that referenced this pull request Jun 18, 2021
* [Citadel] Update tutorials (#204)

Signed-off-by: claireyywang <[email protected]>
Signed-off-by: anindex <[email protected]>

Co-authored-by: anindex <[email protected]>
Co-authored-by: Louise Poubel <[email protected]>
Co-authored-by: Claire Wang <[email protected]>
Co-authored-by: Steve Peters <[email protected]>

* Remove tools/code_check and update codecov (#257)

Signed-off-by: Louise Poubel <[email protected]>

* Fix dart deprecation warning (#263)

Copied from #262.

Signed-off-by: Steve Peters <[email protected]>

Co-authored-by: Claire Wang <[email protected]>
Co-authored-by: anindex <[email protected]>
Co-authored-by: Steve Peters <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏰 citadel Ignition Citadel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants