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

Added Hadrian's Wall Game #463

Closed
wants to merge 1 commit into from
Closed

Conversation

ShilpiBG12
Copy link
Contributor

@ShilpiBG12 ShilpiBG12 commented Oct 31, 2023

Hi,
I have added this new game "Hadrian's Wall". Please see if you merge this contribution against Hacktoberfest. As just 1 more PR need to be completed for completing 4 contributions in Hacktoberfest event, please see if you can merge it fast. Thank you so much for considering. It would be a great help.

Submitting this PR means you have:

Once your PR is created, it will trigger a build a couple of automated checks, including a GitHub Action (Awesome-Lint). This includes a format linting process, so if there are any issues with the build, please address them. The others include a check on the submission limit and also a preview of the site with your changes; please check them out, they are tools tonhelp us both.

Do you have further questions? Check the contribution guidelines or open an issue.

When you're ready, check off the above acknowledgements and continue with your Pull Request... and thanks!

Hi,
I have added this new game "Hadrian's Wall". Please see if you merge this contribution against Hacktoberfest. As just 1 more PR need to be completed for completing 4 contributions in Hacktoberfest event, please see if you can merge it fast.
Thank you so much for considering. It would be a great help.
@ShilpiBG12
Copy link
Contributor Author

@edm00se Sorry to bother you at a short notice. But please see if you can approve this PR under Hacktoberfest. Thanks a lot.

Copy link
Owner

@edm00se edm00se left a comment

Choose a reason for hiding this comment

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

The addition looks fine but this Pull Request will not be accepted as is since your current changes are replacing another entry. Also please re-read the last checkbox provided by the PR template for the description; you checked it, but still @ mentioned me. I will review this PR again later on after you've made your updates to fix the removal of the other game.


### [Twilight Inscription](https://boardgamegeek.com/boardgame/361545/twilight-inscription)
Copy link
Owner

Choose a reason for hiding this comment

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

Do not remove existing entries.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry for this. Will take proper care in future.

@edm00se edm00se added the needs work required updates before a merge can be considered label Oct 31, 2023
@edm00se edm00se mentioned this pull request Oct 31, 2023
6 tasks
@edm00se edm00se closed this Oct 31, 2023
@edm00se
Copy link
Owner

edm00se commented Oct 31, 2023

Closing this one as it's now a duplicate.

@edm00se edm00se added invalid and removed needs work required updates before a merge can be considered labels Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants