Skip to content

refactor: improve code quality and error handling#197

Open
aritra0342 wants to merge 1 commit intopathwaycom:mainfrom
aritra0342:improve-code-quality
Open

refactor: improve code quality and error handling#197
aritra0342 wants to merge 1 commit intopathwaycom:mainfrom
aritra0342:improve-code-quality

Conversation

@aritra0342
Copy link

  • Remove redundant .clone().clone() in time_column.rs
  • Avoid unnecessary Option clone in data_format.rs error message
  • Replace .unwrap() with proper error handling in value.rs
  • Add documentation to parse_pathway_pointer function

These changes improve code clarity and safety without changing behavior.

Introduction

To contribute code to the Pathway project, start by discussing your proposed changes on Discord or by filing an issue.
Once approved, follow the fork + pull request model against the main branch, ensuring you've signed the contributor license agreement.

Context

How has this been tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature or improvement (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issue(s):

Checklist:

  • My code follows the code style of this project,
  • My change requires a change to the documentation,
  • I described the modification in the CHANGELOG.md file.

- Remove redundant .clone().clone() in time_column.rs
- Avoid unnecessary Option clone in data_format.rs error message
- Replace .unwrap() with proper error handling in value.rs
- Add documentation to parse_pathway_pointer function

These changes improve code clarity and safety without changing behavior.
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.

1 participant