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

Run CI Jobs via Github Actions #304

Merged
merged 18 commits into from
Oct 7, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# CODEOWNERS info: https://help.github.com/en/articles/about-code-owners
# Owners are automatically requested for review for PRs that changes code
# that they own.
* @manishrjain @martinmr @jarifibrahim
* @akon-dey @nosql22 @joshua-goldstein @skrdgraph
1 change: 0 additions & 1 deletion .github/ISSUE_TEMPLATE

This file was deleted.

62 changes: 62 additions & 0 deletions .github/ISSUE_TEMPLATE/bug_report.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
name: "🐞 Bug Report"
description: File a bug report
title: "[BUG]: <Title>"
labels: ["kind/bug", "status/triage"]
body:
- type: markdown
attributes:
value: |
Thank you for reporting an issue.
If you suspect this could be a bug, please follow this template.
- type: textarea
attributes:
label: What version of Ristretto are you using?
description: Copy and paste the tag or commit hash used in your application's go.mod file.
placeholder: |
Paste here.
validations:
required: false
- type: textarea
attributes:
label: What version of Go are you using?
description: Copy and paste all you see using the command 'go version'.
placeholder: |
Paste here.
validations:
required: false
- type: dropdown
attributes:
label: Have you tried reproducing the issue with the latest release?
options:
- "Yes"
- "No"
validations:
required: false
- type: textarea
attributes:
label: What is the hardware spec (RAM, CPU, OS)?
description: Share all possible context of your machine.
placeholder: |
Type here.
validations:
required: true
- type: textarea
attributes:
label: What steps will reproduce the bug?
description: Enter details about your bug (command/config used to set up Ristretto).
placeholder: |
Type here.
validations:
required: true
- type: textarea
attributes:
label: Expected behavior and actual result.
description: Enter details about your expectations.
placeholder: |
Type here.
validations:
required: false
- type: textarea
attributes:
label: Additional information
description: Tell us anything else you think we should know.
30 changes: 30 additions & 0 deletions .github/ISSUE_TEMPLATE/documentation_request.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
name: "📃 Documentation Request"
description: Suggest improvements, additions, or revisions to Ristretto documentation
title: "[Documentation]: <Title>"
labels: ["area/documentation", "status/triage"]
body:
- type: markdown
attributes:
value: |
Thank you for reporting an issue.
If you think Ristretto's documentation is lacking, please explain it here.
- type: textarea
attributes:
label: What version of Ristretto is the target?
description: Is it the latest version?
placeholder: |
Type here.
validations:
required: false
- type: textarea
attributes:
label: Documentation.
description: Explain which part of the documentation is lacking.
placeholder: |
Type here.
validations:
required: true
- type: textarea
attributes:
label: Additional information.
description: Tell us anything else you think we should know.
49 changes: 49 additions & 0 deletions .github/ISSUE_TEMPLATE/feature_request.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,49 @@
name: "⚡ Feature Request"
description: Suggest a new feature
title: "[FEATURE]: <Title>"
labels: ["kind/feature ", "status/triage"]
body:
- type: markdown
attributes:
value: |
Thank you for reporting an issue.
If you think we can make Ristretto better, describe your feature request here.
Note: Feature requests are judged based on user experience and modeled on
[Go Experience Reports](https://github.com/golang/go/wiki/ExperienceReports).
These reports should focus on the problems - they should not focus on and need not propose solutions.
- type: dropdown
attributes:
label: Have you tried Ristretto before this proposal? and did not find anything similar?
options:
- "Yes"
- "No"
validations:
required: false
- type: textarea
attributes:
label: What you wanted to do.
description: Enter details about your expectations.
placeholder: |
Type here.
validations:
required: true
- type: textarea
attributes:
label: What you actually did.
description: Enter details about your experience with Ristretto.
placeholder: |
Type here.
validations:
required: false
- type: textarea
attributes:
label: Why wasn't it great, with examples.
description: Enter details about your expectations.
placeholder: |
Type here.
validations:
required: false
- type: textarea
attributes:
label: Additional information.
description: Any external references to support your case.
14 changes: 14 additions & 0 deletions .github/ISSUE_TEMPLATE/question.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
name: "🙋 Question"
description: Question related to Ristretto
title: "[QUESTION]: <Title>"
labels: ["kind/question ", "status/triage"]
body:
- type: markdown
attributes:
value: |
The issue tracker is primarily for code related issues, bug reports, missing documentation, or feature requests.
- If you have a question, please consider asking it on https://discuss.dgraph.io
- type: textarea
attributes:
label: Question.
description: Ask your Question.
29 changes: 29 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
<!--
Change Github PR Title

Your title must be in the following format:
- `topic(Area): Feature`
- `Topic` must be one of `build|ci|docs|feat|fix|perf|refactor|chore|test`

Sample Titles:
- `feat(Enterprise)`: Backups can now get credentials from IAM
- `fix(Query)`: Skipping floats that cannot be Marshalled in JSON
- `perf: [Breaking]` json encoding is now 35% faster if SIMD is present
- `chore`: all chores/tests will be excluded from the CHANGELOG
-->

## Problem
<!--
Please add a description with these things:
1. Explain the problem by providing a good description.
2. If it fixes any GitHub issues, say "Fixes #GitHubIssue".
3. If it corresponds to a Jira issue, say "Fixes DGRAPH-###".
4. If this is a breaking change, please prefix `[Breaking]` in the title. In the description, please put a note with exactly who these changes are breaking for.
-->

## Solution
<!--
Please add a description with these things:
1. Explain the solution to make it easier to review the PR.
2. Make it easier for the reviewer by describing complex sections with comments.
-->
27 changes: 27 additions & 0 deletions .github/workflows/ci-ristretto-lint.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
name: ci-ristretto-lint
on:
push:
branches:
- main
joshua-goldstein marked this conversation as resolved.
Show resolved Hide resolved
pull_request:
branches:
- main
joshua-goldstein marked this conversation as resolved.
Show resolved Hide resolved
schedule:
- cron: "0 * * * *"
jobs:
go-lint:
name: lint
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: golang-lint
env:
# prevent OOM
GOGC: 10
uses: golangci/golangci-lint-action@v2
joshua-goldstein marked this conversation as resolved.
Show resolved Hide resolved
with:
# Required: the version of golangci-lint is required and must be specified without patch version: we always use the latest patch version.
version: v1.36
only-new-issues: true
args: --timeout=10m
skip-go-installation: true
34 changes: 34 additions & 0 deletions .github/workflows/ci-ristretto-tests.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
name: ci-ristretto-tests
on:
push:
branches:
- main
joshua-goldstein marked this conversation as resolved.
Show resolved Hide resolved
pull_request:
branches:
- main
joshua-goldstein marked this conversation as resolved.
Show resolved Hide resolved
schedule:
- cron: "30 * * * *"
jobs:
ristretto-tests:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- name: Get Go Version
run: |
#!/bin/bash
DEFAULT_VERSION="1.17"
GOVERSION=$({ [ -f .go-version ] && cat .go-version; } || echo $DEFAULT_VERSION)
echo "GOVERSION=$GOVERSION" >> $GITHUB_ENV
- name: Setup Go
uses: actions/setup-go@v3
with:
go-version: ${{ env.GOVERSION }}
- name: Run Unit Tests
run: go test -race -covermode atomic -coverprofile=covprofile ./...

Choose a reason for hiding this comment

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

do we need the -race flag here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I took that from the previous test.sh script. Probably it is a good idea to keep it around (only downside is tests may take longer).

Choose a reason for hiding this comment

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

👍

- name: Install Goveralls
run: go install github.com/mattn/goveralls@latest
- name: Send Coverage Results
env:
COVERALLS_TOKEN: ${{ secrets.COVERALLSIO_TOKEN }}
run: goveralls -coverprofile=covprofile

24 changes: 0 additions & 24 deletions .github/workflows/main.yml

This file was deleted.

1 change: 1 addition & 0 deletions .go-version
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
1.17.11
22 changes: 22 additions & 0 deletions .golangci.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
run:
tests: false
skip-dirs:
- contrib

linters-settings:
lll:
line-length: 120

linters:
disable-all: true
enable:
#- errcheck
#- ineffassign
- gas
#- gofmt
#- golint
#- gosimple
#- govet
- lll
#- varcheck
#- unused
7 changes: 3 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,16 +1,15 @@
# Ristretto
[![Go Doc](https://img.shields.io/badge/godoc-reference-blue.svg)](http://godoc.org/github.com/dgraph-io/ristretto)
[![ci-ristretto-tests](https://github.com/dgraph-io/ristretto/actions/workflows/ci-ristretto-tests.yml/badge.svg)](https://github.com/dgraph-io/ristretto/actions/workflows/ci-ristretto-tests.yml)
[![ci-ristretto-lint](https://github.com/dgraph-io/ristretto/actions/workflows/ci-ristretto-lint.yml/badge.svg)](https://github.com/dgraph-io/ristretto/actions/workflows/ci-ristretto-lint.yml)
[![Coverage Status](https://coveralls.io/repos/github/dgraph-io/ristretto/badge.svg?branch=main)](https://coveralls.io/github/dgraph-io/ristretto?branch=main)
[![Go Report Card](https://img.shields.io/badge/go%20report-A%2B-brightgreen)](https://goreportcard.com/report/github.com/dgraph-io/ristretto)
[![Coverage](https://gocover.io/_badge/github.com/dgraph-io/ristretto)](https://gocover.io/github.com/dgraph-io/ristretto)
![Tests](https://github.com/dgraph-io/ristretto/workflows/tests/badge.svg)

Ristretto is a fast, concurrent cache library built with a focus on performance and correctness.

The motivation to build Ristretto comes from the need for a contention-free
cache in [Dgraph][].

**Use [Discuss Issues](https://discuss.dgraph.io/tags/c/issues/35/ristretto/40) for reporting issues about this repository.**

[Dgraph]: https://github.com/dgraph-io/dgraph

## Features
Expand Down
20 changes: 0 additions & 20 deletions test.sh

This file was deleted.