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.

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

[nasa/nos3#403] CI and Repo Cleanup #404

Merged
merged 6 commits into from
Nov 7, 2024
Merged

[nasa/nos3#403] CI and Repo Cleanup #404

merged 6 commits into from
Nov 7, 2024

Conversation

jlucas9
Copy link
Contributor

@jlucas9 jlucas9 commented Nov 1, 2024

Closes #403.

Includes the following improvements:

  • Organized scripts into high level categories
  • Standardize the script naming
  • Removed unnecessary steps from CI
  • Readme updates to include correct directories in use

@jlucas9 jlucas9 self-assigned this Nov 1, 2024
@jlucas9 jlucas9 changed the base branch from main to dev November 1, 2024 10:45
@jlucas9 jlucas9 changed the title WIP: [nasa/nos3#403] CI and Repo Cleanup [nasa/nos3#403] CI and Repo Cleanup Nov 4, 2024
Copy link

@dacarter22 dacarter22 left a comment

Choose a reason for hiding this comment

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

Make prep, make cycle, and make launch all ran.

Copy link
Contributor

@Isgaroth-the-Green Isgaroth-the-Green left a comment

Choose a reason for hiding this comment

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

Are the scripts executable in the branch as-is? I am having to adjust the permissions on nearly all of them to get it to build properly. Also, is there a new version of make real-clean?

Aside from that, everything seems fine; once I made all the scripts executable everything ran.

@jlucas9 jlucas9 merged commit 92c2d3e into dev Nov 7, 2024
2 checks passed
@jlucas9 jlucas9 deleted the nos3#403 branch November 7, 2024 15:39
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.

FSW build not detecting $(FLIGHT_SOFTWARE) env variable
3 participants