Skip to content

Conversation

@nwf
Copy link
Member

@nwf nwf commented May 30, 2020

It has long been superseded by the rfswitch module.

Hold as PR until after next master cut.

@HHHartmann
Copy link
Member

Shouldn't we announce a deprecation first instead of just having a module vanish without warning or explanation?

@HHHartmann HHHartmann mentioned this pull request May 31, 2020
4 tasks
@nwf
Copy link
Member Author

nwf commented May 31, 2020

Well, the documentation already described it as "superseded", but I'm happy to wait for the master drop for this to land.

It has long been superseded by the rfswitch module
@nwf nwf merged commit 61b5407 into nodemcu:dev Jun 10, 2020
@nwf nwf deleted the rm-rc branch June 10, 2020 01:15
@marcelstoer marcelstoer added this to the Next release milestone Jun 10, 2020
@marcelstoer
Copy link
Member

Missing: removal of rc.md from mkdocs.yml. Now fixed.

@nwf
Copy link
Member Author

nwf commented Jun 10, 2020

Hm... This and #3123 passed CI even with the bad mkdocs.yml.

@marcelstoer
Copy link
Member

Yes, but the docs build on RTD failed and that info isn't fed back to GitHub. Not sure if/how that could be achieved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants