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

docs: model installation ADR #390

Merged
merged 2 commits into from
Oct 19, 2023
Merged

docs: model installation ADR #390

merged 2 commits into from
Oct 19, 2023

Conversation

0xSage
Copy link
Contributor

@0xSage 0xSage commented Oct 18, 2023

No description provided.

@0xSage 0xSage changed the title refactor: adr makefile refactor: model installation ADR Oct 18, 2023
@0xSage 0xSage marked this pull request as ready for review October 18, 2023 09:42
@0xSage 0xSage changed the title refactor: model installation ADR docs: model installation ADR Oct 18, 2023
Copy link
Contributor

@louis-jan louis-jan left a comment

Choose a reason for hiding this comment

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

LGTM

usernames=$$(git config user.name); \
sed -i '' 's/{ADR-NUM}/'$$next_number'/g' $(CURDIR)/adr-$$next_number-$$title.md; \
sed -i '' 's/{TITLE}/'$$title'/g' $(CURDIR)/adr-$$next_number-$$title.md; \
sed -i '' 's/{DATE}/'$$date'/g' $(CURDIR)/adr-$$next_number-$$title.md; \
Copy link
Contributor

Choose a reason for hiding this comment

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

Can I ask, which Makefile I should use to create a new adr?, this one looks better than the one defined in the root directory

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the one in root is deleted. very confused for me to have put it in our root dir...

@0xSage 0xSage merged commit a9654ac into main Oct 19, 2023
@0xSage 0xSage deleted the modelinstalladr branch October 19, 2023 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants