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

Update fast track to download .env file since mission model template changed #194

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

dandelany
Copy link
Collaborator

We updated the mission model template to use a .env file but this accidentally broke our Fast Track instructions in the docs. This fixes it by having the user download the .env file too, maybe later we can talk about if there's a more ergonomic way but this gets it working again.

@dandelany dandelany requested a review from skovati October 24, 2024 21:26
Copy link
Contributor

@skovati skovati left a comment

Choose a reason for hiding this comment

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

This matches my idea of a quick fix! Something to consider is maybe pointing users to our mission model template repo / readme

```

If you're running a different OS and do not have [curl](https://curl.se/) available you can [download the docker-compose.yml here](https://raw.githubusercontent.com/NASA-AMMOS/aerie-mission-model-template/main/docker-compose.yml).
If you're running a different OS and do not have [curl](https://curl.se/) available you can [download the docker-compose.yml here](https://raw.githubusercontent.com/NASA-AMMOS/aerie-mission-model-template/main/docker-compose.yml) and [the .env file here](https://raw.githubusercontent.com/NASA-AMMOS/aerie-mission-model-template/refs/heads/main/.env.template) (rename to `.env`).
Copy link
Contributor

Choose a reason for hiding this comment

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

The template does not have the GITHUB_USER and GITHUB_TOKEN specified. Will this cause issues?

Copy link
Contributor

Choose a reason for hiding this comment

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

We shouldn't need it until we need to build models / procedures

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think they're only required for doing a gradle build - testing docker compose up without them seems to work for me

@dandelany dandelany merged commit 3fe252a into develop Oct 25, 2024
1 check passed
@dandelany dandelany deleted the hotfix/fast-track-env branch October 25, 2024 01:10
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