Upgrade to use ElasticSearch 7.0.0

markmcdowell / NLog.Targets.ElasticSearch

NLog target for Elasticsearch

MIT License

176 stars 89 forks source link

Upgrade to use ElasticSearch 7.0.0 #95

Closed ultimaustin closed 5 years ago

ultimaustin commented 5 years ago

Hi, I've made some local changes to upgrade this package to use version 7.0.0 of ElasticSearch.Net (which requires updating the dependencies to minimum .Net 4.7.2). Would I be able to commit this change to a branch for consideration?

markmcdowell commented 5 years ago

Sure, we'd just bump the major version.

ultimaustin commented 5 years ago

Cool. It looks like I don't have rights to check in a new branch or changes to master. Whats the preferred mechanism to submit a change?

markmcdowell commented 5 years ago

You fork the repo to your local, create a branch and then submit a pull request with your changes.

See https://guides.github.com/activities/forking/