Skip to content
This repository was archived by the owner on Apr 24, 2024. It is now read-only.

Rahulbcn27 GitHub actions 1#4

Merged
yadvr merged 21 commits intoapache:mainfrom
rahulbcn27:rahulbcn27-github-actions-1
Jan 30, 2023
Merged

Rahulbcn27 GitHub actions 1#4
yadvr merged 21 commits intoapache:mainfrom
rahulbcn27:rahulbcn27-github-actions-1

Conversation

@rahulbcn27
Copy link
Copy Markdown
Contributor

Migration from Travis CI to Github Actions Support.

@rahulbcn27
Copy link
Copy Markdown
Contributor Author

@rohityadavcloud Please review.

@@ -0,0 +1,29 @@
name: Test-Build
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

missing license @rahulbcn27

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

Copy link
Copy Markdown
Member

@yadvr yadvr left a comment

Choose a reason for hiding this comment

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

missing license

Copy link
Copy Markdown
Member

@yadvr yadvr left a comment

Choose a reason for hiding this comment

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

LGTM - only way to test is to merge this. Merging this.

@yadvr yadvr merged commit 55834cf into apache:main Jan 30, 2023
@yadvr
Copy link
Copy Markdown
Member

yadvr commented Jan 30, 2023

Feedback @rahulbcn27 write better summary in your commit (i.e. commit message) and PR title/description.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants