Skip to content

Remove application name from settings #9

Description

@scottux

I don't think there is a point in having this defined.

Activity

  1. var-foo commented on Sep 30, 2013

    @var-foo
    Member

    no, that's totally useless. We should definitely get rid of that. What's in a name?

  2. scottux commented on Oct 1, 2013

    @scottux
    MemberAuthor

    That which we call a rose by any other name would smell as sweet.

  3. ghost assigned on Oct 1, 2013
  4. scottux commented on Oct 1, 2013

    @scottux
    MemberAuthor

    After digging in, it is used to output the package name in the phpDoc blocks of the generated code. Should we keep this and maybe rename it?* @package <?php echo CodeGen::$ApplicationName; ?>

  5. var-foo commented on Oct 2, 2013

    @var-foo
    Member

    I still don't see a point in keeping it...

  6. scottux commented on Oct 3, 2013

    @scottux
    MemberAuthor

    It would help by auto-documenting the generated code. Even if it isn't necessarily the "Application Name" we could use the variable as "Package Name."

  7. var-foo commented on Oct 4, 2013

    @var-foo
    Member

    I'm fine with that as long as we provide a default. I would imagine that most people don't want to be bothered by this unless we're going to have a config file... or we could think about doing a one-click-install type thing like wordpress does.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions