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

implement observable set #1592

Closed
wants to merge 1 commit into from
Closed

implement observable set #1592

wants to merge 1 commit into from

Conversation

newraina
Copy link

@newraina newraina commented Jun 14, 2018

This is a follow up to the issue: #69

PR checklist:

  • Added unit tests
  • Updated changelog
  • Updated docs (either in the description of this PR as markdown, or as separate PR on the gh-pages branch. Please refer to this PR). For new functionality, at least API.md should be updated
  • Added typescript typings
  • Verified that there is no significant performance drop (npm run perf)

@coveralls
Copy link

coveralls commented Jun 14, 2018

Coverage Status

Coverage decreased (-0.1%) to 93.58% when pulling 7f37b62 on newraina:feature/observable-set into 857621c on mobxjs:master.

@xaviergonz
Copy link
Contributor

@newraina just wondering, what's left to implement this feature?

@newraina
Copy link
Author

@xaviergonz doc and changelog update in the PR checklist... sorry for delay so long

@xaviergonz
Copy link
Contributor

@newraina if that's the only thing that's left I could lend you a hand if you want

@newraina
Copy link
Author

@xaviergonz
OK, I will review my implementation again, handle git conflict, if everything is ok, I'll remove the 'WIP' title. maybe the doc/changelog update should in a separate PR?

@newraina newraina changed the title WIP: implement observable set implement observable set Nov 24, 2018
@mweststrate mweststrate self-assigned this Dec 11, 2018
@mweststrate
Copy link
Member

Apologies for the late follow up! The PR went unnoticed as it seemed the build was still failing. Thanks for the awesome PR and will process it further!

@xaviergonz
Copy link
Contributor

Can't there be a threshold before coveralls marks a PR as failing?

@mweststrate
Copy link
Member

Merging, thanks a lot!

@mweststrate
Copy link
Member

Fixed and released in 4.9.0 / 5.9.0

@newraina newraina deleted the feature/observable-set branch January 22, 2019 06:01
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.

4 participants