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.

Sign up for GitHub

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

Jump to bottom

Switched to direct use of Dockerhub fprime image #895

Merged
merged 152 commits into from
Aug 5, 2021
Merged

Switched to direct use of Dockerhub fprime image #895

merged 152 commits into from
Aug 5, 2021

Conversation

r9-pena
Copy link
Contributor

@r9-pena r9-pena commented Aug 5, 2021

Originating Project/Creator Ricardo Pena
Affected Component CI build test
Affected Architectures(s)
Related Issue(s)
Has Unit Tests (y/n)
Builds Without Errors (y/n) y
Unit Tests Pass (y/n)
Documentation Included (y/n)

Change Description

Using existing Dockerhub fprime image to simplify workflow and improve file extension capabilities.

Rationale

Optimize CI and file usage.

Future Work

Extend feature to Autodocs workflow.

r9-pena and others added 30 commits June 22, 2021 16:00
@r9-pena
Create tutorial-support.yml
366f93f
@r9-pena
Update tutorial-support.yml
3ae4682 
GpsApp tutorial to be pulled for build test and submodules to be pulled recursively
@r9-pena
Merge branch 'nasa:devel' into devel
bc050af
@r9-pena
Implementation of tutorial workflow
9a6e941
@r9-pena
typo in repo address
4078ce3
@r9-pena
Added missing github action file
b0b3806
@r9-pena
Changed workflow path
45e99a1
@r9-pena
Changed dir paths
bce2af6
@r9-pena
Work around for workflow
0811db7
@r9-pena
work around
f70bb74
@r9-pena
troubleshooting
7783654
@r9-pena
troubleshoot
2b47ec7
@r9-pena
troubleshoot
e80529c
@r9-pena
Performance update
0a3d2e6
@r9-pena
Path correction
2103604
@r9-pena
troubleshoot
e03fd06
@r9-pena
Dockerfile correction
15374d0
@r9-pena
path change
6ce47d2
@r9-pena
path change
4c2c09f
@r9-pena
path fix
91f3ab3
@r9-pena
path fix
61191bf
@r9-pena
path fix
c819a78
@r9-pena
path fix
1096697
@r9-pena
paht fix
202100b
@r9-pena
Merge branch 'nasa:devel' into devel
2761184
@r9-pena
Revert to changes
a22de72
@r9-pena
Merge branch 'devel' of https://github.com/r9-pena/fprime into devel
8e7e1ca
@r9-pena
Completed workflow for tutorial support
d6b1a7d
@r9-pena
troubleshoot
e54d12d
@r9-pena
Build path error
9c95406
r9-pena and others added 26 commits August 2, 2021 11:36
@r9-pena
Define entrypoint directly from workflow
afa2cff
@r9-pena
Changed entrypoint method
24388e5
@r9-pena
Changed entrypoint method #2
366a183
@r9-pena
Corrected test path
775d2c6
@r9-pena
Syntax correction
57850a2
@r9-pena
Path Test #1
dc3dcdc
@r9-pena
Path test #2
0ff49c2
@r9-pena
Path Test #4
8f2597d
@r9-pena
Path Test $5
b9e5cdc
@r9-pena
Test Path nasa#6
59fc078
@r9-pena
Path Test nasa#6
0cb5321
@r9-pena
Path Test nasa#7
d65e78f
@r9-pena
Path Test nasa#8
13f5d84
@r9-pena
Refactored scripts to remove master.bash
a924751
@r9-pena
Corrected test path
2ed720e
@r9-pena
Fixed log problems
ba5e293
@r9-pena
Entrypoint alternative
f77cbf5
@r9-pena
Bypassed master.bash file
f9a9c46
@r9-pena
Merge branch 'nasa:devel' into devel
fd6f830
@r9-pena
Added line for better CI error messages
dee313c
@r9-pena
Merge branch 'devel' of https://github.com/r9-pena/fprime into devel
6b079aa
@r9-pena
Merge branch 'nasa:devel' into devel
0f56bbe
@r9-pena
Entrypoint work
a0cb35e 
* Test entrypoint #1

* Test entrypoint #2

* Test entrypoint #3

* Test entrypoint #4

* Test entrypoint #4

* Test entrypoint  nasa#5

* Test entrypoint nasa#6

* Test entrypoint nasa#7

* Test entrypoint nasa#8

* Test entrypoint nasa#9

* Clean up for PR
@r9-pena
Removed commented line
1063e11
@r9-pena
Merge branch 'devel' of https://github.com/nasa/fprime into nasa-devel
f4648e9
@r9-pena
Merge pull request #4 from r9-pena/nasa-devel
a5e8892 
Nasa devel
@LeStarch LeStarch merged commit e7ba8aa into nasa:devel Aug 5, 2021
@LeStarch
Copy link
Collaborator

LeStarch commented Aug 5, 2021

Just out of curiosity....how come we keep seeing the same commits over and over on your PRs? Are you not starting fresh off devel after each PR?

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

@Joshua-Anderson Joshua-Anderson Joshua-Anderson approved these changes

@LeStarch LeStarch LeStarch approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@r9-pena @LeStarch @Joshua-Anderson