alokit-innovations / vibi-dpu Goto Github PK

View Code? Open in Web Editor NEW

6.0 3.0 1.0 673 KB

Vibinex Data Processing Unit for collecting and analyzing private data

Home Page: https://vibinex.com

License: GNU General Public License v3.0

Dockerfile 0.65% Rust 98.11% Python 0.33% Shell 0.92%

bitbucket bitbucket-api code-review docker git github github-api hacktoberfest pubsub pull-requests

vibi-dpu's People

Contributors

Stargazers

Watchers

Forkers

blue0316

vibi-dpu's Issues

We can add a status column to the hunks table in the database which we can update based on the success of this function for easier status report to the client.

          We can add a status column to the hunks table in the database which we can update based on the success of this function for easier status report to the client.

Originally posted by @avikalpg in #6 (comment)

Webhook adding API (among others) not working with gh auth token

File logger doesn't succeed, probably permission issues

Vibinex logo coming on multiple pages like settings and then disappearing

Fix logs in dpu for bitbucket flow

Create a function for getting Bitbucket user from commit

          This whole thing should have been a separate function.

Originally posted by @avikalpg in #29 (comment)

The `PrInfo` struct is being populated directly from the JSON response without any error checking. If the response does not contain the expected fields, this could cause a panic. Consider adding error handling to ensure that the expected fields are present in the response.

          The `PrInfo` struct is being populated directly from the JSON response without any error checking. If the response does not contain the expected fields, this could cause a panic. Consider adding error handling to ensure that the expected fields are present in the response.

let pr_info = match (pr_data["destination"]["commit"]["hash"].as_str(), pr_data["source"]["commit"]["hash"].as_str(), pr_data["state"].as_str(), pr_data["source"]["branch"]["name"].as_str()) {
    (Some(base_head_commit), Some(pr_head_commit), Some(state), Some(pr_branch)) => PrInfo {
        base_head_commit: base_head_commit.to_string(),
        pr_head_commit: pr_head_commit.to_string(),
        state: state.to_string(),
        pr_branch: pr_branch.to_string(),
    },
    _ => {
        eprintln!("Failed to parse PR data: {:?}", pr_data);
        return None;
    }
};

Originally posted by @coderabbitai[bot] in #46 (comment)

Fix login for github app flow and trigger

`deserialize_paginated_webhook_response` seems to be generic and it probably handles the job of `deserialize_paginated_response`

This function seems to be generic and it probably handles the job of deserialize_paginated_response for everyone.

So we can rename this function to the generic version and use it for deserializing the response for the repositories API call as well. That way, we'd be able to resolve the comment in PR #77 as well.

Originally posted by @avikalpg in #79 (comment)

No never nesting?

          No never nesting?

Originally posted by @avikalpg in #11 (comment)

Fix logging for bitbucket flow

Auto assign not working, no login strings for [email protected] found while assigning

All tag related comments to be addressed together in separate PR

          All tag related comments to be addressed together in separate PR

Originally posted by @tapishr in #11 (comment)

Default reviewer assignment

Here is the requirement text from SupplyNote:

Just wanted to ask if there is a feature to set a default approver for each PR along with the one who is relevant.
This might be helpful as it will give visibility to a team lead on all the PRs that are being raised + if relevant user is unavailable,
default approver can review.

We can discuss how do we want to implement this feature.

Fix logs in dpu for github app flow

We are saving this in the db, but are we ever using this value from the db?

          We are saving this in the db, but are we ever using this value from the db?

I think we should use it as the 1st default value in this function itself if config was not sent in the message_data; and default to the RepoConfig::default() only when this information is not available in the db.