Skip to content

Andrehe001 #38

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

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Conversation

andrehe001
Copy link

@andrehe001 andrehe001 commented Dec 2, 2022

Purpose of PR

Type of PR

  • Documentation changes
  • Code changes
  • Test changes
  • CI-CD changes
  • GitHub Template changes
  • Codespaces changes

@@ -7,6 +7,7 @@ key2: Cats
namespace: dogs-cats
nodePort: 30080
port: 8080
targets: []
targets:
Copy link
Contributor

Choose a reason for hiding this comment

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

targets shouldn't be checked in

@@ -5,5 +5,6 @@ image: ghcr.io/cse-labs/pib-imdb:latest
namespace: imdb
nodePort: 30080
port: 8080
targets: []
targets:
Copy link
Contributor

Choose a reason for hiding this comment

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

targets shouldn't be checked in

@@ -0,0 +1,72 @@
dotfiles/placeholder
Copy link
Contributor

Choose a reason for hiding this comment

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

myapp shouldn't be checked in

Copy link
Contributor

@bartr bartr left a comment

Choose a reason for hiding this comment

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

see comments

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