X Tutup
Skip to content

[WIP] Parsing of command-line arguments is broken#66

Draft
Copilot wants to merge 8 commits intomasterfrom
copilot/fix-43
Draft

[WIP] Parsing of command-line arguments is broken#66
Copilot wants to merge 8 commits intomasterfrom
copilot/fix-43

Conversation

Copy link
Contributor

Copilot AI commented May 27, 2025

  • Analyze the issues with the jsonpointer command-line tool
  • Understand the differences between v1 and v2 behavior
  • Modify bin/jsonpointer to handle both use cases:
    • Make first positional argument a file path by adding backward compatibility
    • Add -e/--expression option for direct pointer expressions
    • Fix the -f/--pointer-file option
    • Add proper error handling for file not found case
  • Update the help text to be clear and unambiguous
  • Test the changes with various input combinations
  • Rebase onto latest master (bea0f7e)

⌨️ Start Copilot coding agent tasks without leaving your editor — available in VS Code, Visual Studio, JetBrains IDEs and Eclipse.

Copilot AI and others added 2 commits May 27, 2025 21:23
Co-authored-by: stefankoegl <184196+stefankoegl@users.noreply.github.com>
Co-authored-by: stefankoegl <184196+stefankoegl@users.noreply.github.com>
@stefankoegl
Copy link
Owner

@copilot rebase

Copilot AI and others added 4 commits June 10, 2025 21:22
…ility code

Co-authored-by: stefankoegl <184196+stefankoegl@users.noreply.github.com>
Co-authored-by: stefankoegl <184196+stefankoegl@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jun 10, 2025

@copilot rebase

Rebase completed successfully. The branch has been rebased onto the latest master (602d29a).

@stefankoegl
Copy link
Owner

@copilot rebase

# Conflicts:
#	.github/workflows/test.yaml
#	doc/index.rst
#	setup.py
#	tests.py
Copy link
Contributor Author

Copilot AI commented Mar 23, 2026

@copilot rebase

Rebase completed successfully. The branch has been merged with the latest master (bea0f7e) in commit f9daf3e.

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.

2 participants

X Tutup