Skip to content

align NORTHtool templates with cookiecutter-nomad-plugin#28

Merged
lukaspie merged 12 commits into
mainfrom
north-tool-renaming
Mar 19, 2026
Merged

align NORTHtool templates with cookiecutter-nomad-plugin#28
lukaspie merged 12 commits into
mainfrom
north-tool-renaming

Conversation

@lukaspie
Copy link
Copy Markdown
Contributor

@lukaspie lukaspie commented Mar 17, 2026

Fixes #20

@lukaspie lukaspie marked this pull request as ready for review March 17, 2026 15:10
@lukaspie lukaspie force-pushed the north-tool-renaming branch 3 times, most recently from cd669b9 to c47b054 Compare March 17, 2026 16:29
Copy link
Copy Markdown
Contributor

@RubelMozumder RubelMozumder left a comment

Choose a reason for hiding this comment

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

The only issue is that the tests aren't passing; otherwise, LGTM.

@lukaspie lukaspie force-pushed the north-tool-renaming branch from 98742a4 to 691f12b Compare March 18, 2026 10:10
@mkuehbach mkuehbach self-requested a review March 18, 2026 10:55
Copy link
Copy Markdown
Contributor

@mkuehbach mkuehbach left a comment

Choose a reason for hiding this comment

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

Only one more question, agree with the remainder with Rubel

Copy link
Copy Markdown
Contributor

@RubelMozumder RubelMozumder left a comment

Choose a reason for hiding this comment

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

I found some inconsistencies.

@lukaspie lukaspie merged commit 39ae643 into main Mar 19, 2026
3 checks passed
@lukaspie lukaspie deleted the north-tool-renaming branch March 19, 2026 21:07
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.

Requires update once NOMAD gets a released version

3 participants