Skip to content

Add new boards Phoenix 1.0 & Phoenix 2.0 #2088

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

Merged
merged 4 commits into from
Jun 6, 2016
Merged

Add new boards Phoenix 1.0 & Phoenix 2.0 #2088

merged 4 commits into from
Jun 6, 2016

Conversation

nazt
Copy link
Contributor

@nazt nazt commented Jun 3, 2016

No description provided.

@codecov-io
Copy link

codecov-io commented Jun 3, 2016

Current coverage is 26.31%

Merging #2088 into master will not change coverage

@@             master      #2088   diff @@
==========================================
  Files            19         19          
  Lines          3634       3634          
  Methods         328        328          
  Messages          0          0          
  Branches        585        585          
==========================================
  Hits            956        956          
  Misses         2513       2513          
  Partials        165        165          

Powered by Codecov. Last updated by dd81336...8a1a894

@igrr
Copy link
Member

igrr commented Jun 3, 2016

New boards should also be added to doc/boards.md and package/package_esp8266com_index.template.json

@nazt
Copy link
Contributor Author

nazt commented Jun 4, 2016

@igrr Oh sorry for my fault, I've fixed that

thank you 👍

@igrr
Copy link
Member

igrr commented Jun 4, 2016

Another thing, boards.txt refers to two new variants, but I don't see them added.
phoenix_v2.build.variant=phoenix_v2
phoenix_v1.build.variant=phoenix_v1
So there should be phoenix_v1 and phoenix_v2 subfolders in under variants/ with pins_arduino.h in them.

@nazt
Copy link
Contributor Author

nazt commented Jun 4, 2016

@igrr fixed now! sorry so much.

@igrr igrr merged commit 222e92a into esp8266:master Jun 6, 2016
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.

3 participants