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 fatlum.is-cool.me.json #181

Merged
merged 1 commit into from
Nov 16, 2024
Merged

Conversation

ghost
Copy link

@ghost ghost commented Nov 16, 2024

Requirements

  • You have completed your website.
  • The website is reachable.
  • The CNAME record doesn't contain https:// or /.
  • There is sufficient information at the owner field.

Description

Link to Website

@ghost ghost self-requested a review as a code owner November 16, 2024 08:12
@is-cool-me is-cool-me bot enabled auto-merge November 16, 2024 08:12
Copy link
Contributor

is-cool-me bot commented Nov 16, 2024

Review for Pull Request: Update fatlum.is-cool.me.json

Author: Fatlum

Thank you for contributing to the project! Your efforts are greatly appreciated.

Summary of Changes

The pull request updates the fatlum.is-cool.me.json file by removing the unnecessary TXT record for Discord.

Review

  • The website is complete and reachable.
  • The CNAME record is correctly formatted without https:// or /.
  • The owner field contains sufficient information.

Suggestions

The changes look good overall! Just a quick note to ensure that the removal of the TXT record aligns with your current setup and that it won't affect any existing functionality.

Once again, thank you for your contribution! If you have any questions or need further assistance, feel free to ask.

Best regards,
is-cool-me copilot

@is-cool-me is-cool-me bot merged commit aa6f9fe into is-cool-me:main Nov 16, 2024
8 checks passed
@is-cool-me is-cool-me locked and limited conversation to collaborators Nov 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants