Git Product home page Git Product logo

Comments (6)

inconshreveable avatar inconshreveable commented on June 12, 2024

I don't think there's a change here that will make it better, to be honest. It's not ideal that you need to import the package just to implement the Handler interface, but I think any of the other options are just more complicated and burdensome.

from log15.

ChrisHines avatar ChrisHines commented on June 12, 2024

@inconshreveable Agreed.

I have been thinking about this issue, and I'm not convinced there is a good solution either. One approach is to define the Handler interface solely with built in types or types from the standard library. But that may be painful to use.

This issue can serve to document one of the hidden costs of API changes.

from log15.

notzippy avatar notzippy commented on June 12, 2024

I have had issues with using the versioned form of the URL repo as well. As a solution I have been using a manifest file to pull in the labeled version of the repo directly from github onto my gopath. that works consistent enough for me.

from log15.

ChrisHines avatar ChrisHines commented on June 12, 2024

@notzippy You are talking about gopkg.in/inconshreveable/log15.v2, right? What problems have you had?

from log15.

notzippy avatar notzippy commented on June 12, 2024

@ChrisHines The code completor for intellij insisted that log15 needed to be on the end of the import statement (like import "gopkg.in/inconshreveable/log15.v2/log15"), this is an issue with the tool more then anything else, but I understand why, it is because the package name in the file is not matching log15.v2 which in theory it should.

from log15.

GeertJohan avatar GeertJohan commented on June 12, 2024

@notzippy I think about 30-40% of all non-std packages use a package name that doesn't match the directory they're in.. Lots of packages are prefixed with go. or go-, and then there's also all the gopkg.in versioned packages with the version suffix. The go tool doesn't expect the directory name to be equal to the package name, not does the spec state that this is expected. So I don't think in theory it should and this is definitely a intellij problem.

from log15.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.