Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update README and configuration file #115

Merged
merged 3 commits into from
May 2, 2024
Merged

Conversation

mguetta1
Copy link
Collaborator

No description provided.

@mguetta1 mguetta1 requested a review from aufi April 17, 2024 09:10
@mguetta1 mguetta1 marked this pull request as draft April 17, 2024 09:13
@mguetta1 mguetta1 changed the title Update README and configuration file [WIP] Update README and configuration file Apr 17, 2024
@mguetta1 mguetta1 changed the title [WIP] Update README and configuration file Update README and configuration file Apr 17, 2024
@mguetta1 mguetta1 marked this pull request as ready for review April 17, 2024 09:25
Copy link
Member

@aufi aufi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR Maayan! I'd like ask for more detailed information what applications need what permissions on the token.

Like https://github.com/konveyor/go-konveyor-tests/blob/main/analysis/tc_tackle_testapp_private_binary.go needs token to access XY repository with read access, etc.

Probably don't have to be in readme, comments on the tests case might be enough, just to help find out what token is needed to reproduce the test execution. cc @pranavgaikwad

Signed-off-by: Maayan Hadasi <mguetta@redhat.com>
@mguetta1 mguetta1 force-pushed the readme branch 3 times, most recently from aab5566 to 35fde13 Compare April 30, 2024 10:37
@mguetta1 mguetta1 requested a review from aufi April 30, 2024 10:38
README.md Outdated Show resolved Hide resolved
Signed-off-by: Maayan Hadasi <mguetta@redhat.com>
Copy link
Member

@aufi aufi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks for the PR!

Signed-off-by: Maayan Hadasi <mguetta@redhat.com>
@mguetta1 mguetta1 merged commit 539b876 into konveyor:main May 2, 2024
3 of 5 checks passed
@mguetta1 mguetta1 deleted the readme branch May 2, 2024 10:42
@mguetta1 mguetta1 added the cherry-pick/release-0.3 This PR should be cherry-picked to release-0.3 branch. label May 2, 2024
github-actions bot pushed a commit that referenced this pull request May 2, 2024
Signed-off-by: Maayan Hadasi <mguetta@redhat.com>
Signed-off-by: Cherry Picker <noreply@github.com>
mguetta1 added a commit that referenced this pull request May 2, 2024
Signed-off-by: Maayan Hadasi <mguetta@redhat.com>
Signed-off-by: Cherry Picker <noreply@github.com>
Co-authored-by: Maayan Hadasi <60384172+mguetta1@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick/release-0.3 This PR should be cherry-picked to release-0.3 branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants