Skip to content

Add Issues Templates#193

Merged
emdobrin merged 10 commits intoadobe:devfrom
cacheung:addTemplate
May 10, 2021
Merged

Add Issues Templates#193
emdobrin merged 10 commits intoadobe:devfrom
cacheung:addTemplate

Conversation

@cacheung
Copy link
Copy Markdown
Contributor

@cacheung cacheung commented May 6, 2021

Add Issues Chooser Templates

Description

When users go to Issues and open a new Issue, after this change, it will take to issue chooser page which will have three choices. They are bug report, feature request and Report a security vulnerability.
Open bug report and feature request will bring to a template which have pre-populated format to follow.
Report a security vulnerability will take to the SECURITY.md.

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • 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 change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Add Issues Templates
@cacheung cacheung requested a review from emdobrin May 6, 2021 22:43
@codecov
Copy link
Copy Markdown

codecov bot commented May 6, 2021

Codecov Report

Merging #193 (1b1b410) into dev (a2f07c2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #193   +/-   ##
=======================================
  Coverage   94.29%   94.29%           
=======================================
  Files          26       26           
  Lines         910      910           
=======================================
  Hits          858      858           
  Misses         52       52           

@cacheung cacheung requested a review from nporter-adbe May 6, 2021 22:52
## Prerequisites
<!--- Go through the items below before logging an issue -->
- [ ] Please search this repository's issues to see if it has already been reported. This helps reduce duplicate issues from being created.
- [ ] If this is a Security Disclosure please follow the guidelines in [CONTRIBUTING](../CONTRIBUTING.md). This helps keep folks from accidentally releasing vulnerabilities before the maintainers get a chance to fix the issue.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's replace [CONTRIBUTING](../CONTRIBUTING.md) with [Security Policy](https://github.com/adobe/aepsdk-edge-ios/security/policy)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I changed it pointing to the SECURITY.md file in the root folder

Update with review comments
Add ISSUE_TEMPLATE.md, so open a blank report will pick this one.
Add Security.md
cacheung added 3 commits May 7, 2021 11:33
Update for review comments
Set swift code as default code format
Change the security policy link to relative path
Copy link
Copy Markdown
Contributor

@emdobrin emdobrin left a comment

Choose a reason for hiding this comment

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

Looks good with a tiny comment

@emdobrin emdobrin merged commit 2ba395e into adobe:dev May 10, 2021
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