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 docs for google pages #180

Merged
merged 1 commit into from
Nov 5, 2024
Merged

Update docs for google pages #180

merged 1 commit into from
Nov 5, 2024

Conversation

byronantak
Copy link
Collaborator

  • Update docs README.md for installation steps
  • Update base url to point to the pages instance
  • Update github url to point to correct repo

* Update base url to point pages instance
* Update github url to point to correct repo
@byronantak byronantak requested a review from a team as a code owner November 5, 2024 14:20
@byronantak byronantak enabled auto-merge (squash) November 5, 2024 14:25
@@ -98,7 +98,7 @@ Depending on the file size that will be processed, it might be required to incre
max_allowed_packet = 256M


If using a older version of MySQL, there is a [known issue](https://github.com/box/mojito/issues/120) when creating the schema. One workaround is to use `utf8`
If using a older version of MySQL, there is a [known issue]({{ site.github_url }}/issues/120) when creating the schema. One workaround is to use `utf8`
Copy link
Collaborator

Choose a reason for hiding this comment

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

@byronantak - I don't think that issue id exists in this fork - might need to be updated

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@garionpin Sorry. It put the automerge under the assumption that it worked like azure devops which blocks if there is an unresolved conversation. I fixed it in a follow-up PR
#181

@byronantak byronantak merged commit 2f2ae07 into master Nov 5, 2024
5 checks passed
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.

3 participants