-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
Error handling #73
Open
ManoloTonto1
wants to merge
68
commits into
arran4:master
Choose a base branch
from
ManoloTonto1:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Error handling #73
Changes from all commits
Commits
Show all changes
68 commits
Select commit
Hold shift + click to select a range
08adbda
Add Todo and Journal Property Methods
spslater 14095cc
Add Alarm, Timezone, and FreeBusy Property Methods
spslater 67727dd
Fix Typos in a couple places
spslater b51e64d
Merge branch 'arran4:master' into add-todo-journal
spslater 6d4f14f
Add Comment and Category Properties to Components
spslater a5a215d
Merge branch 'master' into add-todo-journal
spslater 4b2d6a1
Add changes for All Day Events to All Day Todos
spslater 23c4714
added errors.go and converted errors to constants for better error ha…
ManoloTonto1 b69874f
fixed some namings
ManoloTonto1 7d97e96
Bump gopkg.in/yaml.v3 from 3.0.0-20210107192922-496545a6307b to 3.0.0
dependabot[bot] ce8cff1
Merge pull request #76 from arran4/dependabot/go_modules/gopkg.in/yam…
arran4 0abebc2
Add VEvent.GetDtStampTime
galenwarren acf49e9
Merge pull request #80 from coachclientconnect/add-get-timestamp
arran4 a0f11e3
add possibility to remove VEvent from calendar by id
zachmann 70f69c8
undo formatting change
zachmann 1dfe73f
remove redundant return statement
zachmann 7678c37
Merge pull request #81 from zachmann/master
arran4 e69ce9c
Add ComponentPropertyPriority; the VTODO component can have that
quite aef0a29
Merge pull request #82 from quite/add-component-priority
arran4 8525f6b
Add {Get,Set}LastModifiedAt
bcspragu 1a61754
Merge pull request #83 from bcspragu/last-modified
arran4 a55aac2
Merge remote-tracking branch 'origin/pull-66-add-todo-journal-orphan'…
arran4 5f3bef9
Unnecessary functions causing lint recursion errors.
arran4 55df13e
Consistent todo receiver
arran4 9e30bdf
Ignore explicitly
arran4 1e96c15
Consistent receivers.
arran4 cf8d1b3
Ignore explicitly
arran4 804f4d3
Merge pull request #84 from arran4/pull-66-add-todo-journal
arran4 681cc6e
fix panic for missing property param operator
brackendawson 3631125
fix panic when property ends without colon or value
brackendawson a8f0586
fix panic when param value has incomplete escape sequence
brackendawson 46e2a5c
Exclude fuzz testing from pre-1.18 toolchains
brackendawson 51fa6f1
Merge pull request #85 from brackendawson/panic
arran4 3ffa099
fix: only escape property values when serializing.
frereit 0f8a325
fix: reorder string replaces in escaping values.
frereit 647cf9e
test: Add test for escaped semicolons in property parameters
frereit 542d6e5
Merge pull request #86 from JM-Lemmi/fix/text_escaping
arran4 7bd8708
test: Add test for escaped semicolons in RRULEs
frereit bb39d64
Revert "fix: only escape property values when serializing."
frereit 0fcebed
Merge pull request #87 from JM-Lemmi/fix/text_escaping
arran4 f5b4408
Prefix mailto: for email in organizer property
meain 93ca35f
Only conditionally add in the mailto: prefix for attendee and organizer
meain 84a339a
Merge pull request #88 from meain/org-mailto
arran4 02c6335
Re-add Calendar.RemoveEvent method
zachmann 9fab3f0
Merge pull request #90 from zachmann/master
arran4 1e5b6e4
add: contributing.md
brenank 27bb2dd
fix: simplify & update ghas
brenank 909bd3b
fix: explicitly list golang versions
brenank 3e4aba3
fix: use macos-13, arm is not supported by setup-go
brenank f044c04
Merge pull request #92 from brenank/brenank/update-golang
arran4 008589d
add: stable serialization of property parameters
brenank baa1c1d
add: serialiation test
brenank ea37e62
fix: VFREEBUSY serialization
brenank 8d1cec9
breaking: unescape Property.Value of type TEXT
brenank 68b453c
refactor: fix linting errors
brenank b866620
refactor: reduce scope of gitignore
brenank 2f726f2
refactor: switch syntax
brenank 9a99514
fix: add test err assertion
brenank 20061ba
fix: reduce build restriction on serialization test
brenank 6591742
Merge pull request #91 from brenank/master
arran4 c331f7a
Add ComponentPropertyRelatedTo; the VTODO component can have that
quite ea4ce01
Merge pull request #94 from quite/add-component-relatedto
arran4 f76e1f3
fixed merge conflicts
ManoloTonto1 e616cd6
fixed some namings
ManoloTonto1 2520cb0
Merge branch 'master' of github.com:ManoloTonto1/golang-ical
ManoloTonto1 4c3688b
fixed som functions that are using the wrong Struct Embeddings
ManoloTonto1 1d9da2e
Some suggestions, this could be improved still. (#1)
arran4 3a306dd
Error handling conflicts again (#2)
arran4 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,39 @@ | ||
--- | ||
name: 🐞 Bug | ||
about: File a bug/issue | ||
title: '[BUG] <title>' | ||
labels: Bug, Needs Triage | ||
assignees: '' | ||
|
||
--- | ||
|
||
<!-- | ||
Note: Please search to see if an issue already exists for the bug you encountered. | ||
--> | ||
|
||
### Current Behavior: | ||
<!-- A concise description of what you're experiencing. --> | ||
|
||
### Expected Behavior: | ||
<!-- A concise description of what you expected to happen. --> | ||
|
||
### Steps To Reproduce: | ||
<!-- | ||
Example: steps to reproduce the behavior: | ||
1. In this environment... | ||
1. With this config... | ||
1. Run '...' | ||
1. See error... | ||
--> | ||
|
||
### Minimal Example ical extract: | ||
|
||
```ical | ||
BEGIN:VCALENDAR | ||
.... | ||
``` | ||
|
||
### Anything else: | ||
<!-- | ||
Links? References? Anything that will give us more context about the issue that you are encountering! | ||
--> |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
--- | ||
name: Something else | ||
about: Any other issue | ||
title: '<title>' | ||
labels: Needs Triage | ||
assignees: '' | ||
|
||
--- | ||
|
||
<!-- | ||
Note: Please search to see if an issue already exists for the bug you encountered. | ||
|
||
Please include smallest possible sized examples. | ||
--> | ||
|
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,35 @@ | ||
<!-- | ||
|
||
Thanks for contributing! | ||
|
||
--> | ||
|
||
# Pull Request Template | ||
|
||
## Description | ||
|
||
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. | ||
|
||
<!-- The following line will automatically close the issues if done correctly --> | ||
|
||
Fixes # (issue) | ||
|
||
## Type of change | ||
|
||
Please delete options that are not relevant. | ||
|
||
- [ ] Bug fix (non-breaking change which fixes an issue) | ||
- [ ] New feature (non-breaking change which adds functionality) | ||
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) -- This might take a while to be considered | ||
|
||
## Must haves: | ||
|
||
- [ ] I have commented in hard-to-understand areas and anywhere the function not immediately apparent | ||
- [ ] I have made corresponding changes to the comments (if any) | ||
- [ ] My changes generate no new warnings | ||
- [ ] I have added tests that prove the fix is effective or that my feature works | ||
- [ ] I have added tests that protects the code from degradation in the future | ||
|
||
## Nice to haves: | ||
|
||
- [ ] I have added additional function comments to new or existing functions |
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
name: gitleaks | ||
on: [push,pull_request] | ||
jobs: | ||
gitleaks: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v1 | ||
- name: gitleaks-action | ||
uses: zricethezav/gitleaks-action@master | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
name: Go vunderability check | ||
on: [push, pull_request] | ||
jobs: | ||
build: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Setup Golang | ||
uses: actions/setup-go@v5 | ||
- id: govulncheck | ||
uses: golang/govulncheck-action@v1 |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,28 +1,39 @@ | ||
on: [push, pull_request] | ||
name: Test | ||
jobs: | ||
test: | ||
version: | ||
name: Test | ||
permissions: | ||
contents: read | ||
strategy: | ||
matrix: | ||
go-version: [1.14.x, 1.15.x, 1.16.x, 1.17.x] | ||
os: [ubuntu-latest, macos-latest, windows-latest] | ||
go-version: ['oldstable', 'stable'] | ||
os: ['ubuntu-latest', 'macos-13', 'windows-latest'] | ||
runs-on: ${{ matrix.os }} | ||
steps: | ||
- name: Install Go | ||
uses: actions/setup-go@v2 | ||
- name: Checkout | ||
uses: actions/checkout@v4 | ||
- name: Setup Golang | ||
uses: actions/setup-go@v5 | ||
with: | ||
go-version: ${{ matrix.go-version }} | ||
- name: Cache-Go | ||
uses: actions/cache@v1 | ||
go-version: "${{ matrix.go-version }}" | ||
- name: Go Test | ||
run: go test -race ./... | ||
module: | ||
name: Test | ||
permissions: | ||
contents: read | ||
strategy: | ||
matrix: | ||
go-version-file: ['go.mod'] | ||
os: ['ubuntu-latest', 'macos-13', 'windows-latest'] | ||
runs-on: ${{ matrix.os }} | ||
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v4 | ||
- name: Setup Golang | ||
uses: actions/setup-go@v5 | ||
with: | ||
path: | | ||
~/go/pkg/mod # Module download cache | ||
~/.cache/go-build # Build cache (Linux) | ||
~/Library/Caches/go-build # Build cache (Mac) | ||
key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }} | ||
restore-keys: | | ||
${{ runner.os }}-go-${{ hashFiles('**/go.sum') }} | ||
- name: Checkout code | ||
uses: actions/checkout@v2 | ||
- name: Test | ||
run: go test ./... | ||
go-version-file: "${{ matrix.go-version-file }}" | ||
- name: Go Test | ||
run: go test -race ./... |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,2 @@ | ||
/.idea/ | ||
/testdata/serialization/actual | ||
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
# Contributing | ||
|
||
## Linting | ||
Make sure your code has been linted using [golangci-lint](https://github.com/golangci/golangci-lint?tab=readme-ov-file#install-golangci-lint) | ||
|
||
```shell | ||
$ golangci-lint run | ||
``` | ||
|
||
## Tests | ||
|
||
If you want to submit a bug fix or new feature, make sure that all tests are passing. | ||
```shell | ||
$ go test ./... | ||
``` | ||
|
||
|
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
Oops, something went wrong.
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.
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.
Not necessary?