Skip to content

LANTERN-866: Removed db fetcher function and all calls.#522

Closed
daedalus-s wants to merge 1 commit intomainfrom
LANTERN-866-remove-database-fetcher
Closed

LANTERN-866: Removed db fetcher function and all calls.#522
daedalus-s wants to merge 1 commit intomainfrom
LANTERN-866-remove-database-fetcher

Conversation

@daedalus-s
Copy link
Copy Markdown
Collaborator

@daedalus-s daedalus-s commented Apr 24, 2025

Pull requests into this repository require the following checks to be completed by the submitter and reviewers.

Submitter:

  • Administrative tasks are complete.
    • This pull request describes why these changes were made.
    • The JIRA ticket links to this PR.
    • The JIRA ticket for this PR is: LANTERN-866
  • Tests are complete.
    • Tests are included and test edge cases.
    • Tests have been run locally and pass.
  • Code review has been performed.
    • Code diff has been done and checked to ensure only expected code is being committed.
    • A linter for the language has been run.

Primary Reviewer:

  • Code works.
    • Code has been run locally and the basic functionality affected by the PR has been checked.
    • Edge cases have been checked to ensure behavior is as expected.
  • Code review has been performed.
    • Code is maintainable, reusable, efficient, and correct.
    • Code accomplishes the tasks purpose.
    • Code follows style guidance appropriate for the language, including passing linter checks.
    • Code is well commented (which does not mean verbosely commented).
  • Tests are complete.
    • Tests are included and test edge cases.
    • Tests have been run locally and pass.

Merger:

Prior to merging, ensure that all checks have been completed. Delete the branch after the merge
is complete.

@rishi-salunkhe-mettle
Copy link
Copy Markdown
Contributor

This PR has been merged as part of the following PR: #535

Closing this PR.

@rishi-salunkhe-mettle rishi-salunkhe-mettle deleted the LANTERN-866-remove-database-fetcher branch May 20, 2025 18:33
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.

3 participants