Skip to content

ci: lint, test and coverage report#1028

Merged
nitisht merged 3 commits into
parseablehq:mainfrom
de-sh:main
Dec 10, 2024
Merged

ci: lint, test and coverage report#1028
nitisht merged 3 commits into
parseablehq:mainfrom
de-sh:main

Conversation

@de-sh
Copy link
Copy Markdown
Contributor

@de-sh de-sh commented Dec 9, 2024

To help keep track of test coverage in the repo

Description


This PR has:

  • been tested to ensure log ingestion and log query works.
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added documentation for new or modified features or behaviors.

@de-sh
Copy link
Copy Markdown
Contributor Author

de-sh commented Dec 9, 2024

Please enable coveralls.io against the repo

@de-sh de-sh requested a review from nitisht December 9, 2024 10:49
Comment thread README.md Outdated
Comment thread README.md Outdated
Devdutt Shenoi added 2 commits December 9, 2024 20:22
Signed-off-by: Devdutt Shenoi <devdutt@outlook.in>
@nitisht nitisht merged commit d1c6644 into parseablehq:main Dec 10, 2024
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