Skip to content

refactor(slots): Improve Slots Implementations#92

Merged
faustbrian merged 3 commits intoArkEcosystemArchive:masterfrom
sleepdefic1t:refactor/improve-slot-implementations
May 26, 2019
Merged

refactor(slots): Improve Slots Implementations#92
faustbrian merged 3 commits intoArkEcosystemArchive:masterfrom
sleepdefic1t:refactor/improve-slot-implementations

Conversation

@sleepdefic1t
Copy link
Contributor

Proposed changes

This PR improves the Slots implementations for OS & IoT builds.
As a result of this improvement and decoupling, an additional dependency is able to be removed.

Specifically, this PR does the following:

  • Further decouples OS code from IoT code.
  • Adds missing ESP8266 functionality.
  • Allows removal of lib/STL dependency.
    • STL lib is only included to provide a conv. method for IoT to use date.h
    • date.h is now only used by OS builds, which already have the STL.
  • updated change log.
  • added PIO filters for lib/date.

Types of changes

  • Refactoring (improve a current implementation without adding a new feature or fixing a bug)

Checklist

  • I have read the CONTRIBUTING documentation
  • Lint and unit tests pass locally with my changes

- Further decouple OS code from IoT code.
- Add missing ESP8266 functionality.
- Allow removal of lib/STL dependency.
  - STL lib is only included to provide a conv. method for IoT to use date.h
  - date.h is now only used by OS builds, which will have the STL by default.
- update changelog.
- add PIO filters for lib/date.
-
@ghost ghost added Complexity: High More than 256 lines changed. Type: Refactor The pull request improves or enhances an existing implementation. labels May 21, 2019
@sleepdefic1t
Copy link
Contributor Author

additional passing tests @:

@sleepdefic1t
Copy link
Contributor Author

@faustbrian
This is ready.
Not sure why CI hasn’t run here.
All tests pass on Appveyor and Circle (links above).

@sleepdefic1t sleepdefic1t mentioned this pull request May 24, 2019
3 tasks
@faustbrian faustbrian merged commit 48e014c into ArkEcosystemArchive:master May 26, 2019
@sleepdefic1t sleepdefic1t deleted the refactor/improve-slot-implementations branch May 27, 2019 11:38
@sleepdefic1t sleepdefic1t restored the refactor/improve-slot-implementations branch May 27, 2019 11:39
@sleepdefic1t sleepdefic1t deleted the refactor/improve-slot-implementations branch May 27, 2019 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Complexity: High More than 256 lines changed. Type: Refactor The pull request improves or enhances an existing implementation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants