Skip to content

Conversation

@mguludag
Copy link
Member

@mguludag mguludag linked an issue Apr 11, 2025 that may be closed by this pull request
@mguludag mguludag force-pushed the 9-provide-github-devcontainer-support branch from b5520fa to 67bd53b Compare April 11, 2025 19:07
@mguludag mguludag self-assigned this Apr 11, 2025
@mguludag mguludag added enhancement New feature or request infra labels Apr 11, 2025
Copy link
Collaborator

@RaduNichita RaduNichita left a comment

Choose a reason for hiding this comment

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

Hey @mguludag, many thanks for the contribution! It looks pretty ok to me, could you please a small section in the README.md about how to use .devcontainer, please?

},
"github-cli": "latest"
}
} No newline at end of file
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
}
}

Copy link
Member Author

Choose a reason for hiding this comment

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

done

steps:
- name: Deploy to GitHub Pages
id: deployment
uses: actions/deploy-pages@v4 No newline at end of file
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
uses: actions/deploy-pages@v4
uses: actions/deploy-pages@v4

Copy link
Member Author

Choose a reason for hiding this comment

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

done

name: Deploy to GitHub Pages
needs: build

# Grant GITHUB_TOKEN the permissions required to make a Pages deployment
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you put a TODO here, please?

Copy link
Member Author

Choose a reason for hiding this comment

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

added

@mguludag mguludag force-pushed the 9-provide-github-devcontainer-support branch from 67bd53b to 072c768 Compare April 12, 2025 16:14
@mguludag
Copy link
Member Author

Hey @mguludag, many thanks for the contribution! It looks pretty ok to me, could you please a small section in the README.md about how to use .devcontainer, please?

Instructions for devcontainer is added into README.md

@mguludag mguludag force-pushed the 9-provide-github-devcontainer-support branch from 072c768 to 160481b Compare April 12, 2025 22:30
@netlify
Copy link

netlify bot commented Apr 12, 2025

Deploy Preview for bemanproject ready!

Name Link
🔨 Latest commit 160481b
🔍 Latest deploy log https://app.netlify.com/sites/bemanproject/deploys/67fae97476496900089918e3
😎 Deploy Preview https://deploy-preview-29--bemanproject.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@RaduNichita RaduNichita merged commit 0ecc4b7 into main Apr 13, 2025
4 checks passed
@mguludag mguludag deleted the 9-provide-github-devcontainer-support branch April 18, 2025 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request infra

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provide GitHub devcontainer support

3 participants