Skip to content

Move contents of project_api to export/__init__.py #3997

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 1 commit into from
Mar 29, 2017

Conversation

theotherjimmy
Copy link
Contributor

@theotherjimmy theotherjimmy commented Mar 22, 2017

The file project_api.py was poorly named and caused much confusion.
Given that it was actually a front end to the exporters, I put it into
the export sub-dir.

Testing

  • /morph export-build : in case I broke something

@theotherjimmy
Copy link
Contributor Author

@bridadan @c1728p9 More tech-debt related stuff.

The file `project_api.py` was poorly named and caused much confusion.
Given that it was actually a front end to the exporters, I put it into
the export sub-dir.
Copy link
Contributor

@bridadan bridadan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks lovely 👍

@bridadan
Copy link
Contributor

/morph export-build

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph export-build

Output

mbed Build Number: 139

All exports and builds passed!

@theotherjimmy
Copy link
Contributor Author

yeah! good to go.

@bridadan
Copy link
Contributor

Gonna run the basic test too just to make sure we don't gunk up the other PRs.

/morph test

@theotherjimmy
Copy link
Contributor Author

Sounds good.

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1730

Test failed!

@theotherjimmy
Copy link
Contributor Author

huh. That's strange.

@bridadan
Copy link
Contributor

It's a timing test bug caused by CI machine load. We should really look at this again @c1728p9 @studavekar

/morph test

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1736

All builds and test passed!

@sg- sg- merged commit 432162f into ARMmbed:master Mar 29, 2017
@theotherjimmy theotherjimmy deleted the remove-project-api branch March 29, 2017 21:43
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.

6 participants