Skip to content

Python3 warning for dev branch Readme - #148

Merged
redragonx merged 7 commits into
NaomiProject:naomi-devfrom
AustinCasteel:dev
Jan 11, 2019
Merged

redragonx merged 7 commits into
NaomiProject:naomi-devfrom
AustinCasteel:dev

Conversation

@AustinCasteel

Copy link
Copy Markdown
Member

Description

Until the 3.0 release, there needs to be a notice in the readme that states there was a change so new contributors are made aware before developing

Related Issue

Specific issue for this PR
Context issue for this PR

Motivation and Context

If someone updates their existing Naomi installation without running the new setup script to install the proper changes, Naomi breaks. Between the changes already made and bringing attention to the change on the readme there would be multiple places denoting the change, lessening the chance of someone inadvertently breaking Naomi.

How Has This Been Tested?

An alert image was added to the website directory that is then implemented into the readme for display.

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:

  • 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 added tests to cover my changes.
  • All new and existing tests passed.

@redragonx redragonx left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thanks @Longshotpro2

@redragonx
redragonx merged commit fe59d60 into NaomiProject:naomi-dev Jan 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants