Skip to content

Change alphabetise argument in moo filter commands - #1090

Merged
Matthew Mizielinski (matthew-mizielinski) merged 2 commits into
v4.0_releasefrom
1089_nc-alphabetise-fix
Sep 15, 2026
Merged

Matthew Mizielinski (matthew-mizielinski) merged 2 commits into
v4.0_releasefrom
1089_nc-alphabetise-fix

Conversation

@matthew-mizielinski

Copy link
Copy Markdown
Collaborator

Closes #1089

A fairly trivial fix if this works.

@matthew-mizielinski Matthew Mizielinski (matthew-mizielinski) changed the title nc alphabetise fix Change alphabetise argument in moo filter commands Sep 8, 2026
@matthew-mizielinski
Matthew Mizielinski (matthew-mizielinski) changed the base branch from main to v4.0_release September 8, 2026 13:02
@matthew-mizielinski
Matthew Mizielinski (matthew-mizielinski) marked this pull request as ready for review September 11, 2026 09:51
@matthew-mizielinski

Copy link
Copy Markdown
Collaborator Author

I've tested this and run the tests and all works fine.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me and all tests pass

Matthew Mizielinski (matthew-mizielinski) added a commit that referenced this pull request Sep 15, 2026
* #1089: alter alphabetisation argument in moo filter commands
@matthew-mizielinski
Matthew Mizielinski (matthew-mizielinski) deleted the 1089_nc-alphabetise-fix branch September 15, 2026 09:46
@matthew-mizielinski

Copy link
Copy Markdown
Collaborator Author

Cherry picked to main in d36cab7

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.

ncks -a option is deprecated

2 participants