-
Notifications
You must be signed in to change notification settings - Fork 208
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
Changed discord as the primary way to get in touch with the comminuty #1279
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
robacarp
approved these changes
May 17, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great start
robacarp
added a commit
that referenced
this pull request
May 25, 2022
* Update README.md * Delete FUNDING.yml * Changed discord as the primary way to get in touch with the comminuty (#1279) * Changed discord as the primary way to get in touch with the comminuty * Changed the discord link so that it uses a permanent invite link * Updated the version number in Amber to reflect the actual release number (#1280) * Changed discord as the primary way to get in touch with the comminuty * Changed the discord link so that it uses a permanent invite link * Fixed the version number being out dated from the actual latest release number. * Fix the correct documentation url (#1278) Co-authored-by: Philippe Nénert @papilip <me@papilip.me> * Feature/version upgrade (#1282) * Changed discord as the primary way to get in touch with the comminuty * Changed the discord link so that it uses a permanent invite link * Fixed the version number being out dated from the actual latest release number. * Updated Amber version and default Granite version * Feature/update readme contribution steps (#1283) * Changed discord as the primary way to get in touch with the comminuty * Changed the discord link so that it uses a permanent invite link * Fixed the version number being out dated from the actual latest release number. * Update Linux installation steps to reflect the actual steps needed to install from source Co-authored-by: Isaac Sloan <isaac@isaacsloan.com> Co-authored-by: robacarp <robacarp@users.noreply.github.com> Co-authored-by: Philippe Nénert <me+github@papilip.me> Co-authored-by: Philippe Nénert @papilip <me@papilip.me>
crimson-knight
added a commit
that referenced
this pull request
Mar 28, 2023
* Update README.md * Delete FUNDING.yml * Changed discord as the primary way to get in touch with the comminuty * Changed the discord link so that it uses a permanent invite link * Changed discord as the primary way to get in touch with the comminuty (#1279) * Changed discord as the primary way to get in touch with the comminuty * Changed the discord link so that it uses a permanent invite link * Fixed the version number being out dated from the actual latest release number. * Updated the version number in Amber to reflect the actual release number (#1280) * Changed discord as the primary way to get in touch with the comminuty * Changed the discord link so that it uses a permanent invite link * Fixed the version number being out dated from the actual latest release number. * Fix the correct documentation url (#1278) Co-authored-by: Philippe Nénert @papilip <me@papilip.me> * Feature/version upgrade (#1282) * Changed discord as the primary way to get in touch with the comminuty * Changed the discord link so that it uses a permanent invite link * Fixed the version number being out dated from the actual latest release number. * Updated Amber version and default Granite version * Feature/update readme contribution steps (#1283) * Changed discord as the primary way to get in touch with the comminuty * Changed the discord link so that it uses a permanent invite link * Fixed the version number being out dated from the actual latest release number. * Update Linux installation steps to reflect the actual steps needed to install from source * Make default package support older and newer versions of node-sass * Bump amber version and update Ameba version * Added github actions for CI instead of TravisCI * Amber version bump for these quality of life updates * Fix naming warning raised by crystal compiler about positional names being mismatched * removed the travisCI yml * Fixed amber_spec cookie naming error that relied on old implicit encoding/decoding of cooke name & value * Fixed areas that ameba caught for format improvements * Changed ubuntu to an older version instead of latest * Changed ci workflow to remove an extra sqlite3 dev library that may be causing issues * Changed from uninstall to install to fix sqlite3 issue * Changed installation of sqlite3 based on further reading * Set apt install to automatically agree to prompts * changed packed installed from full sqlite3 to sqlite3-dev * Adds redis for the test suite * Added redis dependencies to the apt install step * Changed steps to stop using the supercharge redis installation and added a step to check that redis is running * Trying redis as a service * Tweaked the redis service settings to map the port for the host container to access redis using localhost:6379 * Added a step to check access to the redis service before the tests run * redis-cli stopped existing so trying to install plain redis * removed the port mapping to test if redis becomes accessible from the runner * specified the host when testing the redis connection * Added ports mapping back to the redis service container to see if this resolves the redis-cli not reaching the redis service * Changed host from localhost to redis to see if that maps correctly * Added the REDIS_URL with the hostname of the redis service and removed the redis connection verification step now that it's working * Changed the empty? method to be more clear about 'any origin' being the actual question, and fixed the to_h method doing too much to return a hash * Remove crystalshards.xyz link on template view * Include the discord link into the generated home page templates * Tweaked how the PG test data is configured * fix to cli scaffold controller spec deprecated garnetspec parent class * Updated the remaining two templates that used the old GarnetSpec parent class to use the new/correct parent class * Updated docker file to use latest crystal image and updated stragling Amber version * Changed node version check to look for v14 only * Bump amber version and updated cli template to use latest release version instead of master branch * added "/myapp" to be excluded from git tracking * Added cli and cli.dwarf to the git ignore so less clean-up is required when working on the cli tool * Changed PG to check for the current user and omit the password for a speedier way to get started --------- Co-authored-by: Isaac Sloan <isaac@isaacsloan.com> Co-authored-by: robacarp <robacarp@users.noreply.github.com> Co-authored-by: Philippe Nénert <me+github@papilip.me> Co-authored-by: Philippe Nénert @papilip <me@papilip.me> Co-authored-by: Spenser Coke <work@spensercoke.com> Co-authored-by: Rafael Revi <rafael.e.revi@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updated the README link to refer new users to the discord server